[Pkg-mol-devel] [Pkg-mol-commits] r152 - in mol/trunk/debian: . debian.mol-source

Gaudenz Steinlin gaudenz at debian.org
Wed Sep 26 21:39:59 UTC 2007


Hi Jörg

I'm a bit late to the party, but anyway some comments to your changes. I
really appreciate most of them! Thank you for working on MOL. If plan to
do further work you can also add yourself to the Uploaders field. So it
will be more visible that you are part of the team and the package will
apear on your qa.debian.org page. Sorry, this mail got rather long...

Some general remarks:
- Thanks for the general overhaul of the debian/rules file. Most of it
  still is from the time when Jens maintained the package and much cruft
  accumulated over the years... The new file looks much cleaner.
- Please always also update the debian/changelog when commiting changes
  (in the same commit)
- I'd rather prefer small commits which only change one thing at the
  time. But this is a minor thing and I also didn't follow that rule all 
  the time in the past.

Answers to the questions in your previous mail:
* Why is a ~ in the version? ~ is a special character. I would use +

This is on purpose exactly because ~ is special. I used .dfsg before and
the problem was, that when upstream introduced a 4th component in the
version number d (the start of dfsg) sorted after 1 (the new version
number component). I wanted to avoid this problem in the future. + would
also be ok, but what if upstream decides to use + in a version number?

* Did upstream refuse to include the manpages?

No, although I'm not sure if I already sent everything upstream.

* Does upstream know the problem with the buggy distclean, i.e. it
  doesn't remove all files and it runs configure.
* Why does upstream ignore the values (e.g. --datadir) passed to
  configure?
* Why does upstream build the configuration interface if defconf is
  used?

IMHO the upstream build system is a complete uncomprehensible mess. I
already struggeld hours with it. I don't know if the current upstream
(Joe Jezak) plans to replace it. Most of the build system originates
from the previous upstream author and creator of mol. Do you have the
time to talk to upstream about these things? Joe is generally busy but
quite responsive and nice. I expect that he would gladly accept patches.

* lintian complains about hyphen-used-as-minus-sign in manpages
* The copyright files is not up to date

Feel free to fix the manpages and copyright file ;-)


Building the current svn HEAD in pbuilder fails with the following
error. Please always test with pbuilder before commiting. The build
works if you change the invocation back to
KERNEL_SOURCE=/lib/modules/`uname -r`/build  $(MAKE) -C src/kmod/Linux/
setup-common

Don't ask me why this matters...

= Building mol-img               
make[1]: Leaving directory `/tmp/buildd/mol-0.9.72.1~dfsg'
/usr/bin/make -C src/kmod/Linux/ KERNEL_SOURCE=/lib/modules/`uname -r`/source \
            setup-common
make[1]: Entering directory `/tmp/buildd/mol-0.9.72.1~dfsg/src/kmod/Linux'
Kernel source missing. KERNEL_SOURCE=/lib/modules/2.6.23-rc6/source
make[1]: *** [sanity-test] Error 1
make[1]: Leaving directory `/tmp/buildd/mol-0.9.72.1~dfsg/src/kmod/Linux'
make: *** [build-stamp] Error 2
pbuilder: Failed autobuilding of package


On Tue, Sep 25, 2007 at 01:51:54PM +0000, jo-guest at alioth.debian.org wrote:
> Author: jo-guest
> Date: 2007-09-25 13:51:54 +0000 (Tue, 25 Sep 2007)
> New Revision: 152
> 
> Added:
>    mol/trunk/debian/debian.mol-source/changelog.m4
> Removed:
>    mol/trunk/debian/debian.mol-source/changelog.m4.m4
>    mol/trunk/debian/mol-source.dirs
> Modified:
>    mol/trunk/debian/Makefile.mol-source
>    mol/trunk/debian/control
>    mol/trunk/debian/debian.mol-source/rules
>    mol/trunk/debian/mol.dirs
>    mol/trunk/debian/mol.menu
>    mol/trunk/debian/rules
> Log:
> ?\226?\128?\162 debian/control
>   ?\194?\183 Removed m4 from the build dependencies; it's replaced by sed.

OK. You should also remove the line below the sed call from
debian/rules. See my comment below in the diff.

> 
> ?\226?\128?\162 debian/mol-source.dirs
>   ?\194?\183 Removed. It's not necessary.

OK. I see, because you changed install into dh_install this works. This is 
clearly a good thing (tm)! We could also use mol-source.install and 
mol.install and also install the lintian overrides this way.

> 
> ?\226?\128?\162 debian/mol.dirs
>   ?\194?\183 Removed all directories they aren't necessary. Most of them are created by
>     other debhelpers.
These are probably OK.
> 
> ?\226?\128?\162 debian/mol.menu
>   ?\194?\183 Adapted to the new menu scheme.

Thanks!

> 
> ?\226?\128?\162 debian/rules
>   ?\194?\183 Added -g to the CFLAGS because policy section 10.1 suggests to build with
>     debugging symbols and strip them later

OK

> 
>   ?\194?\183 Added support for DEB_BUILD_OPTIONS=noopt as suggested by policy
>     section?\194?\16010.1.

OK

> 
>   ?\194?\183 Rearranged the targets in the file: clean, build, binary.

OK, if you think it's better like this. I don't mind. Please place the
"rm -r modules" call in the clean target and not in the binary-arch
target. This makes it easier to inspect the result after the build (eg.
if something goes wrong).

> 
>   ?\194?\183 clean:
>     + Removed the dh_testroot. clean doesn't need root previleges.

Root privileges are not needed, you are correct. The advantage of
calling dh_testroot here is that the build with dpkg-buildpackage or
debuild fails earlier if you forget to call them with fakeroot. If you
think it's better to remove the call I don't really mind.

> 
>     + Put the unpatch target into the command sequence, because it's more
>       intuitive.

OK, I don't mind. I just followed the advice in the dpatch README.

> 
>     + Removed debian/tmp from the dh_clean call. It's not needed.

OK

> 
>     + Call ?\226?\128?\156dpatch apply Makefile?\226?\128?\157 before running make distclean, because it
>       fails due to the missing bootx directory.

OK. That's why there was a - in front of $(MAKE) clean. The reasoning
was something like if the source is not patched yet we did not build and
don't have to clean anyway... Your way of doing it is obviously more
robust.

> 
>   ?\194?\183 Merged the configure and the build targets.

I'd rather keep them separate. This seems to be common practice. Is
there any advantage in merging them.

> 
>   ?\194?\183 Use make defconf instead of yes '' | make conf to create the
>     configuration.

OK, sounds good. I always wondered why Jens put this funny construct
there. 
BTW did you ever check the pci-proxy support or the debugger? What's
your opinion on enabling these featuers in the Debian package?

> 
>   ?\194?\183 binary-arch:
>     + Added a -a to every dh_* call, because we build the arch specific
>       packages.

sounds reasonable.

> 
>     + No need to fix manually the permission of mol.symbols

OK

> 
>     + Replaced the m4 call by an sed call. This saves a build dependency. The
>       file should get renamed, because sed can edit inplace.

OK

> 
>   ?\194?\183 Added the get-orig-source target that fetches the upstream tar.bz2 and
>     creates the .orig.tar.gz. This is what policy suggests in section 4.9.

Having this target is a good thing. But I don't like the way you
duplicate code that is in the "make_orig_tarballs" script in svn. Please
use this script in this target!

One reason why I did not yet implement this target myself is, that mol
is not the usual situtation where you just want to delete some non-free
parts from the source, but you want to split the upstream tarball into
several source packages. So this does not really belong into this source
package any more than into the source of mol-drivers-macos(x).

But I'm open to suggestion of better handling of the source tarballs.
How I currently handle things is documented in debian/README and
debian/README.Debian-source.

> 
> ?\226?\128?\162 debian/Makefile.mol-source
>   ?\194?\183 Made the make calls in build and install targets sensible for the exit
>     code. Fixes #412756

Great! 

> 
> ?\226?\128?\162 debian/debian.mol-source/changelog.m4 debian/debian.mol-source/changelog.m4.m4
>   ?\194?\183 Rename changelog.m4.m4 to changelog.m4 after the switch from m4 to sed in
>     debian/rules.

OK

> 
> ?\226?\128?\162 debian/debian.mol-source/rules
>   ?\194?\183 Made the build of files from m4 files more generic?\226?\128?\148it's make.

OK, I don't mind as long as it works.

> 
>   ?\194?\183 Removed the targets kdist_changes, source and diff, because they do
>     nothing except telling to don't use them.

OK, as long as make-kpkg still works without them. I don't know how long
these things are obsolete now and what the originally did. Probably this
is a leftover from Years ago...

> 
> 
> Modified: mol/trunk/debian/Makefile.mol-source
> ===================================================================
> --- mol/trunk/debian/Makefile.mol-source	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/Makefile.mol-source	2007-09-25 13:51:54 UTC (rev 152)
> @@ -32,13 +32,13 @@
>  .PHONY:	$(SUBDIRS)
>  build:	$(SUBDIRS)
>  $(SUBDIRS):
> -	-$(MAKE) -C $(KSRC) M=`pwd`/$@
> +	$(MAKE) -C $(KSRC) M=`pwd`/$@
>  
>  # descend into subdirectories and install the modules
>  .PHONY: $(SUBDIRS:=-install)
>  install: build $(SUBDIRS:=-install)
>  $(SUBDIRS:=-install): 
> -	-$(MAKE) INSTALL_MOD_PATH=$(DESTDIR) -C $(KSRC) M=`pwd`/$(subst -install,,$@) modules_install
> +	$(MAKE) INSTALL_MOD_PATH=$(DESTDIR) -C $(KSRC) M=`pwd`/$(subst -install,,$@) modules_install
>  
>  # descend into subdirectories and clean up
>  .PHONY:	$(SUBDIRS:=-clean)
> 
> Modified: mol/trunk/debian/control
> ===================================================================
> --- mol/trunk/debian/control	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/control	2007-09-25 13:51:54 UTC (rev 152)
> @@ -3,7 +3,7 @@
>  Priority: optional
>  Maintainer: MOL Packaging Team <pkg-mol-devel at lists.alioth.debian.org>
>  Uploaders: Sven Luther <luther at debian.org>, Otavio Salvador <otavio at debian.org>, Gaudenz Steinlin <gaudenz at debian.org>, Aurélien GÉRÔME <ag at roxor.cx>
> -Build-Depends: debhelper (>= 5), binutils-dev, bison, docbook-to-man, flex, imagemagick, libasound2-dev, libesd0-dev, libncurses5-dev, po-debconf, libxt-dev, libxxf86dga-dev, dpatch, bzip2, m4, zlib1g-dev, autoconf
> +Build-Depends: debhelper (>= 5), binutils-dev, bison, docbook-to-man, flex, imagemagick, libasound2-dev, libesd0-dev, libncurses5-dev, po-debconf, libxt-dev, libxxf86dga-dev, dpatch, bzip2, zlib1g-dev, autoconf
>  Standards-Version: 3.7.2
>  XS-Vcs-Svn: svn://svn.debian.org/pkg-mol/mol/trunk
>  XS-Vcs-Browser: http://svn.debian.org/wsvn/pkg-mol/mol/trunk
> 
> Copied: mol/trunk/debian/debian.mol-source/changelog.m4 (from rev 151, mol/trunk/debian/debian.mol-source/changelog.m4.m4)
> ===================================================================
> --- mol/trunk/debian/debian.mol-source/changelog.m4	                        (rev 0)
> +++ mol/trunk/debian/debian.mol-source/changelog.m4	2007-09-25 13:51:54 UTC (rev 152)
> @@ -0,0 +1,6 @@
> +mol-modules-KVERS (MOL_VERSION+KDREV) unstable; urgency=low
> +
> +  * Built from the mol-modules-source package.
> +
> + -- KMAINT <KEMAIL>  DEBDATE
> +
> 
> Deleted: mol/trunk/debian/debian.mol-source/changelog.m4.m4
> ===================================================================
> --- mol/trunk/debian/debian.mol-source/changelog.m4.m4	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/debian.mol-source/changelog.m4.m4	2007-09-25 13:51:54 UTC (rev 152)
> @@ -1,6 +0,0 @@
> -mol-modules-KVERS (MOL_VERSION+KDREV) unstable; urgency=low
> -
> -  * Built from the mol-modules-source package.
> -
> - -- KMAINT <KEMAIL>  DEBDATE
> -
> 
> Modified: mol/trunk/debian/debian.mol-source/rules
> ===================================================================
> --- mol/trunk/debian/debian.mol-source/rules	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/debian.mol-source/rules	2007-09-25 13:51:54 UTC (rev 152)
> @@ -42,7 +42,7 @@
>  
>  	$(MAKE) clean
>  
> -	-dh_clean
> +	dh_clean
>  	rm -f debian/control debian/changelog
>  	rm -f configure-stamp build-stamp install-stamp
>  
> @@ -57,10 +57,8 @@
>  
>  	touch $@
>  
> -debian/control: debian/control.m4
> +debian/%: debian/%.m4
>  	$(M4) $(M4DEFINES) $< > $@
> -debian/changelog: debian/changelog.m4
> -	$(M4) $(M4DEFINES) $< > $@
>  
>  binary-mol-modules: install-stamp
>  	dh_testdir
> @@ -81,9 +79,6 @@
>  
>  kdist_configure: configure-stamp
>  
> -kdist_changes:
> -	@echo >&2 'Note: kdist_changes is obsolete'
> -
>  kdist_clean: clean
>  
>  kdist_image:
> @@ -102,7 +97,4 @@
>  binary binary-arch:
>  	@echo >&2 'Use make-kpkg to compile this module' && false
>  
> -source diff:
> -	@echo >&2 'source and diff are obsolete - use dpkg-source -b'; false
> -
>  .PHONY: binary binary-arch source diff
> 
> Deleted: mol/trunk/debian/mol-source.dirs
> ===================================================================
> --- mol/trunk/debian/mol-source.dirs	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/mol-source.dirs	2007-09-25 13:51:54 UTC (rev 152)
> @@ -1 +0,0 @@
> -usr/src
> 
> Modified: mol/trunk/debian/mol.dirs
> ===================================================================
> --- mol/trunk/debian/mol.dirs	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/mol.dirs	2007-09-25 13:51:54 UTC (rev 152)
> @@ -1,11 +1,3 @@
> -etc/mol
> -usr/bin
> -usr/lib/mol
> -usr/share/doc/mol
>  usr/share/lintian/overrides
> -usr/share/man/man1
> -usr/share/man/man5
> -usr/share/man/man7
>  usr/share/pixmaps
> -var/lib/mol
>  var/state/mol
> 
> Modified: mol/trunk/debian/mol.menu
> ===================================================================
> --- mol/trunk/debian/mol.menu	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/mol.menu	2007-09-25 13:51:54 UTC (rev 152)
> @@ -1,12 +1,12 @@
>  ?package(mol): \
>  	needs="text" \
>  	command="startmol" \
> -	section="Apps/Emulators" \
> +	section="Applications/Emulators" \
>  	title="Mac-on-Linux (Mac OS 9 and earlier)" \
>  	icon="/usr/share/pixmaps/mol.xpm"
>  ?package(mol):	\
>  	needs="text" \
>  	command="startmol -X" \
> -	section="Apps/Emulators" \
> +	section="Applications/Emulators" \
>  	title="Mac-on-Linux (Mac OS X)" \
>  	icon="/usr/share/pixmaps/mol.xpm"
> 
> Modified: mol/trunk/debian/rules
> ===================================================================
> --- mol/trunk/debian/rules	2007-09-24 08:13:08 UTC (rev 151)
> +++ mol/trunk/debian/rules	2007-09-25 13:51:54 UTC (rev 152)
> @@ -8,148 +8,140 @@
>  # include dpatch rules
>  include /usr/share/dpatch/dpatch.make
>  
> -# Debian specific flags to make,, mostly installation directories
> -DEBIANFLAGS=\
> -prefix=/usr \
> -datadir=/usr/share/mol \
> -libdir=/usr/lib/mol \
> -docdir=/usr/share/doc/mol
> +# the values from configure aren't used by the Makefile so we have to pass
> +# them to every make call
> +DEBIANFLAGS=prefix=/usr datadir=/usr/share/mol libdir=/usr/lib/mol \
> +  docdir=/usr/share/doc/mol
>  
> +CFLAGS += -g -D_FILE_OFFSET_BITS=64
> +
> +ifneq (,$(findstring noopt,$(DEB_BUILD_OPTIONS)))
> +	CFLAGS += -O0
> +else
> +	CFLAGS += -O2
> +endif
> +
>  # Debian package version
>  VERSION=$(shell dpkg-parsechangelog | grep Version: | cut -d " " -f 2)
> +UPSTREAM_VERSION=$(shell echo $(VERSION) | sed s/~dfsg.*//)
>  
> -configure: configure-stamp
> -configure-stamp: patch
> +clean:
>  	dh_testdir
> +	dh_clean
> +	rm -f build-stamp
>  
> -# configure the package.
> -	CFLAGS="$$CFLAGS -D_FILE_OFFSET_BITS=64" ./configure --disable-png
> -	yes '' | make config
> +# remove the manpages
> +	rm -f debian/*.[17] mol.tar.bz2
>  
> -	touch configure-stamp
> +	dpatch apply Makefile
> +	$(MAKE) distclean
>  
> -build: configure build-stamp
> -build-stamp:
> +# upstream bug: distclean does not remove these files
> +	rm -rf obj-ppc config/configure
> +	rm -f .config* util/img/mol-img
> +
> +	$(MAKE) -f debian/rules unpatch
> +
> +build: build-stamp
> +build-stamp: patch
>  	dh_testdir
>  
>  # build the Debian specific manpages
>  	(cd debian; \
>  		for man in mol.7 moldeb.1 molrcget.1 strip_nwrom.1; \
> -		do docbook-to-man `echo $$man|sed 's/[17]/sgml/'` >$$man; \
> +		do docbook-to-man $${man%?}sgml >$$man; \
>  	done)
>  
> +	./configure --disable-png CFLAGS="$(CFLAGS)"
> +	make defconfig
> +
>  # build the package
>  	$(MAKE) BUILD_MODS=n NETDRIVER=n $(DEBIANFLAGS)
>  
>  # setup the build structure for the kernel modules
> -	KERNEL_SOURCE=/lib/modules/`uname -r`/build  $(MAKE) -C src/kmod/Linux/ setup-common
> +	$(MAKE) -C src/kmod/Linux/ KERNEL_SOURCE=/lib/modules/`uname -r`/source \
> +	    setup-common
>  	$(MAKE) -C src/netdriver/ setup-tree-common
>  
> -	touch build-stamp
> +	touch $@
>  
> -clean: clean-patched unpatch
> -clean-patched:
> -	dh_testdir
> -	dh_testroot
> -	rm -f configure-stamp build-stamp
> +# Build architecture-independent files here.
> +binary-indep:
> +# We have nothing to do by default.
>  
> -# remove the manpages
> -	rm -f debian/*.[17]
> -
> -# clean the package, including auto-generated stuff that distclean overlooks
> -	-$(MAKE) clean
> -	rm -rf obj-ppc
> -	rm -f .config*
> -	rm -rf config/configure
> -
> -#	cd src/vconfig; rm -f modes-y.c modes-y.h modes-l.c
> -#	rm -f src/shared/config.h.in
> -#	rm -f src/kmod/mpc107/Makefile src/cpu/mpc107/Makefile src/cpu/ppc/asm_offsets.h
> -#	rm -f mollib/mol.symbols
> -#	cd config/kconfig; rm -rf zconf-y.c zconf-y.h zconf-l.c .autoinclude
> -
> -# remove the modules source directory and tarball
> -	rm -rf modules
> -	rm -f mol.tar.bz2
> -
> -	dh_clean debian/tmp
> -
> -install: build
> +# Build architecture-dependent files here.
> +binary-arch: build
>  	dh_testdir
>  	dh_testroot
> -	dh_clean -k
> -	dh_installdirs
>  
> +	dh_installdirs -a
> +
>  # install the overrides file for lintian
> -	install -o root -g root -m 644 debian/lintian.overrides `pwd`/debian/mol/usr/share/lintian/overrides/mol
> +	install -o root -g root -m 644 debian/lintian.overrides \
> +	    debian/mol/usr/share/lintian/overrides/mol
>  
>  # install the Mac-on-Linux emulator into mol.
>  	$(MAKE) DESTDIR=`pwd`/debian/mol $(DEBIANFLAGS) install
>  
> -# fix permissions
> -	chmod 644 `pwd`/debian/mol/usr/lib/mol/mol.symbols
> +	rm debian/mol/usr/share/doc/mol/COPYING
>  
> -# remove gratious GPL license text
> -	rm `pwd`/debian/mol/usr/share/doc/mol/COPYING
> +	convert -depth 8 -size 100x122 -frame 14x3 -mattecolor '#ffffce' \
> +	    -scale 25% rgb:`pwd`/mollib/graphics/bootlogo.raw \
> +	    debian/mol/usr/share/pixmaps/mol.xpm
>  
> -# extract a 32x32 XPM menu logo from the raw boot logo
> -	convert -depth 8 -size 100x122 -frame 14x3 -mattecolor '#ffffce' -scale 25% rgb:`pwd`/mollib/graphics/bootlogo.raw `pwd`/debian/mol/usr/share/pixmaps/mol.xpm
> -
>  # put together the mol-source package by copying the relevant code
> -	mkdir -p modules/mol/kmod modules/mol/netdriver
> -	-cp -L obj-ppc/build/src/kmod/* modules/mol/kmod
> -	-cp -L obj-ppc/build/src/netdriver/* modules/mol/netdriver
> +	mkdir -p modules/mol
> +# the directory is full of symlinks
> +	cp -prL obj-ppc/build/src/kmod obj-ppc/build/src/netdriver \
> +	    modules/mol
>  
>  # add the Debian directory and the top-level Makefile
>  	cp -pr debian/debian.mol-source modules/mol/debian
> -	m4 -DMOL_VERSION=$(VERSION) < modules/mol/debian/changelog.m4.m4 > modules/mol/debian/changelog.m4
> +	sed -i '1s/MOL_VERSION/$(VERSION)/' modules/mol/debian/changelog.m4

The line below should be deleted as well or the build will fail here.
>  	rm modules/mol/debian/changelog.m4.m4
>  	cp -p debian/Makefile.mol-source modules/mol/Makefile
>  
>  # set correct permissions
>  	chmod u+x modules/mol/debian/rules
> -	chown -R root.src modules
> +	chown -R root:src modules
>  	chmod -R u+w,go-w,a+rX modules
>  
> -# create a tarball
>  	tar jcf mol.tar.bz2 modules
> +	rm -r modules
> +	dh_install -pmol-source mol.tar.bz2 usr/src
>  
> -# install the source tarball into mol-source
> -	install -o root -g root -m 644 mol.tar.bz2 `pwd`/debian/mol-source/usr/src
> +	dh_installdebconf -a
> +	dh_installdocs -a
> +	dh_installexamples -a
> +	dh_installman -a
> +	dh_installmenu -a
> +	dh_installmodules -a
> +	dh_installchangelogs -a CHANGELOG
> +	dh_link -a
> +	dh_strip -a
> +	dh_compress -a
> +	dh_fixperms -a
> +	dh_installdeb -a
> +	dh_shlibdeps -a -Xlib/modules
> +	dh_gencontrol -a
> +	dh_md5sums -a
> +	dh_builddeb -a
>  
> -# create the mol-modules packages by moving the installed modules
> -#	dh_movefiles --sourcedir=debian/mol
> -# remove the now empty directory
> -#	rm -rf `pwd`/debian/mol/lib
> +binary: binary-indep binary-arch
>  
> -# Build architecture-independent files here.
> -binary-indep: build install
> -# We have nothing to do by default.
> +TAR_NAME=mol-$(UPSTREAM_VERSION).tar.bz2
> +MOL_DIR_NAME=mol-$(shell echo $(VERSION) | sed s/-.*//)
>  
> -# Build architecture-dependent files here.
> -binary-arch: build install
> -	dh_testdir
> -	dh_testroot
> -	dh_installdebconf	
> -	dh_installdocs
> -	dh_installexamples
> -	dh_installman
> -	dh_installmenu
> -	dh_installmodules
> -	dh_installchangelogs CHANGELOG
> -	dh_link
> -	dh_strip
> -	dh_compress
> -	dh_fixperms
> -	dh_installdeb
> -	dh_shlibdeps -Xlib/modules
> -	dh_gencontrol
> -	dh_md5sums
> -	dh_builddeb
> +get-orig-source:
> +	wget http://dfn.dl.sourceforge.net/sourceforge/mac-on-linux/$(TAR_NAME)
> +	tar -xf $(TAR_NAME)
> +	rm $(TAR_NAME)
>  
> -# Any other binary targets build just one binary package at a time.
> -binary-%: build install
> -        make -f debian/rules binary-common DH_OPTIONS=-p$*
> +	mv $(TAR_NAME:.tar.bz2=) $(MOL_DIR_NAME)
> +	cd $(MOL_DIR_NAME) && rm -r bootx mollib/drivers \
> +	    mollib/images/moldisk.dmg mollib/images/moldiskX.dmg
>  
> -binary: binary-indep binary-arch
> -.PHONY: build clean binary-indep binary-arch binary install
> +	tar -czf $(MOL_DIR_NAME).orig.tar.gz $(MOL_DIR_NAME)
> +	rm -r $(MOL_DIR_NAME)
> +
> +.PHONY: build clean binary-indep binary-arch binary get-orig-source
> 
> 
> _______________________________________________
> Pkg-mol-commits mailing list
> Pkg-mol-commits at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-mol-commits

-- 
Ever tried. Ever failed. No matter.
Try again. Fail again. Fail better.
~ Samuel Beckett ~



More information about the Pkg-mol-devel mailing list