[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: New speakup-udeb component

Frans Pop, le Sat 31 May 2008 13:41:16 +0200, a écrit :
> On Wednesday 28 May 2008, Samuel Thibault wrote:
> > Possibly not, see attached files: speakup-udeb.startup gets run early,
> > sees the speakup statement on the kernel command line, and in that
> > case modprobes speakup and disables the framebuffer.  In addition to
> > that, speakup-udeb.debinst selects the text frontend.  Eventually,
> > speakup-udeb.finish-install installs the module on the target system,
> > and sets the module to be auto-loaded on reboot.
> Where exactly were you installing these files (full path/filenames 
> please)?

speakup-udeb.finish-install to /usr/lib/finish-install.d/05speakup
speakup-udeb.debinst to /lib/debian-installer.d/S20speakup
speakup-udeb.startup to /lib/debian-installer-startup.d/S37speakup

> > The scripts by themselves should be fine. The problem is more about
> > room.  Speakup modules need a total of 160KB. If that's fine with any
> > installation medium, then great, we can just always enable it. We could
> > also always have the script, and include the modules only in the cases
> > where we can afford it.
> Right. As the scripts themselves are negligible, I think we should just 
> include them by default and determine actual support by including or not 
> including the kernel module udebs. We'll have to see for what arches and 
> images we can do that.
> What is the status of the kernel support and the l-m-e packaging?

Attached is the patch to l-m-e which I have used.

> > Scrips will just not work when the modules are not available.
> Well, that does not seem to be entirely accurate: they will work, but 
> their effect will be nil.

Right :)

> It seems to me that the 2 later script should 
> check that the $SYNTH module was actually loaded successfully.

We can replace the cmdline parsing by lsmod and that should work fine

> From the finish-install script:
> >	if apt-install speakup-modules 1>&2
> Will that actually work?

I couldn't test indeed it since the package wasn't yet on the mirrors.

> Should that not include the kernel version, ABI and flavor?

Hum, indeed.  I don't know how to do that properly.

Index: speakup/copyright
--- speakup/copyright	(révision 0)
+++ speakup/copyright	(révision 0)
@@ -0,0 +1,6 @@
+ Speakup is licensed under the GPL; the GNU General Public License.
+On Debian systems, the complete text of the GNU General Public License
+can be found in `/usr/share/common-licenses/GPL'.
Index: speakup/rules
--- speakup/rules	(révision 0)
+++ speakup/rules	(révision 0)
@@ -0,0 +1,7 @@
+	$(MAKE) -C $(HEADERS_DIR) M=$(CURDIR)/$(DIR)/src $(JOBS_ARG) `cat $(CURDIR)/$(DIR)/src/allmodule.mk`
+	touch $@
+install: LIB_MODULES = $(PACKAGE_DIR)/lib/modules/$(REAL_VERSION)
+	$(MAKE) -C $(HEADERS_DIR) M=$(CURDIR)/$(DIR)/src modules_install INSTALL_MOD_PATH=$(PACKAGE_DIR) INSTALL_MOD_DIR=extra/$(MODULE) `cat $(CURDIR)/$(DIR)/src/allmodule.mk`
Index: speakup/defines
--- speakup/defines	(révision 0)
+++ speakup/defines	(révision 0)
@@ -0,0 +1,7 @@
+not-featuresets: smp
+desc: A screen review module for the Linux kernel
+ Speakup allows you to interact with applications and the GNU/Linux
+ operating system with audible feedback from the console using a
+ synthetic speech device.
Index: defines
--- defines	(révision 11250)
+++ defines	(copie de travail)
@@ -14,6 +14,7 @@
+ speakup

Reply to: