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

Re: [MoM] ProbABEL packaging



On Wed, Dec 11, 2013 at 11:19:25PM +0100, L.C. Karssen wrote:
> >> For the extIDS.pl file I would like to solve the problem by installing
> >> it in the examples directory (that's where it actually belongs, it's not
> >> required for regular use) instead of /usr/bin/. To this end I want to
> >> patch the src/Makefile.am file, which I would then send upstream.
> >>
> >> I've set up dquilt according to sections 3.1 and 3.2 of
> >> http://www.debian.org/doc/manuals/maint-guide/modify.en.html.
> >> Next I took the following steps (as outlined in the URL):
> >>   dquilt new fix-extIDS.pl.patch
> >>   dquilt add src/Makefile.am
> >>  <edit src/Makefile.am>
> >>   dquilt refresh
> >>   dquilt header -e
> >>  <edit patch info>
> >>
> >> However, neither debuild, nor pdebuild seems to pick up the patch. The
> >> patch itself seems ok:

Sorry, what is "dquilt"  - I only know quilt (without the d in the
beginning)?

The problem you were facing is that you did commit the patch and the
patched file to the git repository.  The trouble to distinguish this is
one reasons why people here sometimes blame the quilt format as
suboptimal when using git.  So before you commit your changes you need
to do a

    quilt pop -a

to ensure that you are working on a clean upstream source when commiting
your patches.  I restored the original upstream source in commit

   f102ee1d3141be3bba95e16d90f59bc5e9f9d01c

I also have used `cme fix dpkg-control` (see

   e1c851706593ea122cfb1240c639f7670408e141

) which I (strongly) recommend to anybody doing some packaging today.  It
just does "the right thing" (tm) with your d/control file. :-)

Finally I commited a change which actually brings your change into
effect because without doing a reconfigure Makefile.am is ignored and
configure is just used as is (at least if no specific tricks are done
which I did not checked for).

By doing so I tried to build the package again but I was running into

make  check-TESTS
make[3]: Entering directory `/tmp/buildd/probabel-0.4.1/checks'
make[4]: Entering directory `/tmp/buildd/probabel-0.4.1/checks'
/bin/bash: /bin/bash: cannot execute binary file
make[4]: *** [check_probabel.pl_chunk.sh.log] Error 126
make[4]: Leaving directory `/tmp/buildd/probabel-0.4.1/checks'
make[3]: *** [check-TESTS] Error 2
make[3]: Leaving directory `/tmp/buildd/probabel-0.4.1/checks'
make[2]: *** [check-am] Error 2
make[2]: Leaving directory `/tmp/buildd/probabel-0.4.1/checks'
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory `/tmp/buildd/probabel-0.4.1'
dh_auto_test: make -j1 check returned exit code 2
make: *** [build] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2


So it seems the change has introduced a problem with the reconfiguration
since if you do

$ git diff
diff --git a/debian/rules b/debian/rules
index 4ece2fa..970e910 100755
--- a/debian/rules
+++ b/debian/rules
@@ -17,7 +17,7 @@ GIT_TAG        := $(subst ~,_,$(VERSION))
 # a similar manner are welcome.
 
 %:
-       dh $@  --parallel --with autoreconf
+       dh $@  --parallel
 
 override_dh_installdocs:
        dh_installdocs


the build works again.

Hope this gives some useful hints

      Andreas.


-- 
http://fam-tille.de


Reply to: