mc patch system
Chow Loong Jin
hyperair at gmail.com
Mon Apr 13 21:53:33 BST 2009
On Mon, 2009-04-13 at 13:16 -0700, Brian Murray wrote:
> On Mon, Apr 13, 2009 at 09:59:54PM +0200, Emilio Pozuelo Monfort wrote:
> > Brian Murray wrote:
> > > I've recently been looking at bugs with patches and ran across
> > > http://launchpad.net/bugs/333835 which seemed easy enough to incorporate
> > > and a worthwhile fix for Jaunty. However, I ran into a problem trying
> > > to get the existing patches in mc to apply and adding that particular
> > > patch. Is anyone familiar with how the patch system in mc works?
> > I haven't looked at it, but it build-depends on quilt so I guess it uses it.
> > Here's a small howto:
> > http://pkg-perl.alioth.debian.org/howto/quilt.html
> Maybe I should have elaborated a bit more. While it uses quilt
> executing 'quilt push -a' fails with the message 'No patches in series'.
> Additionally, the structure of the debian/patches directory seemed a bit
> odd to me:
> all.series bugs mc.ext README utf8
> autgen-run.patch debian po syntax vfs
I've just read through the debian/rules file, and found this rule:
apply-patches: pre-build debian/stamp-patched
@echo "patches: $(DEB_PATCHES)"
@set -e ; \
if [ ! "$@" = "reverse-patches" ]; then \
ln -s debian/patches/all.series series;\
ln -s debian/patches/ patches; \
$(QUILT) --quiltrc /dev/null push -a; \
$(QUILT) --quiltrc /dev/null pop -a || true \
rm -f series patches; \
rm -rf .pc; \
if [ "$@" = "debian/stamp-patched" ]; then touch debian/stamp-patched; fi
Specifically, what you want to look at is that it links
debian/patches/all.series to ./series, and debian/patches to ./patches.
Hence you can't use the usual export QUILT_PATCHES=debian/patches way.
You'll have to set QUILT_PATCHES=patches, and make the appropriate
ln -s debian/patches patches
ln -s debian/patches/all.series ./series
Then just use quilt as usual. When you're done:
quilt pop -a
rm -rf patches series .pc
P.S, That's one hell of a convoluted rules file, and I'm glad I'm not
the maintainer of the package. Colin's build system... looks like half
of CDBS crammed into one debian/rules file.
Chow Loong Jin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/ubuntu-motu/attachments/20090413/743fb911/attachment.pgp
More information about the Ubuntu-motu