Package: quilt
Version: 0.67+really0.66-1
Severity: normal
X-Debbugs-Cc: tj.iam...@proton.me

It is confusing that despite the man-page describing the environment
over-rides that can be set either on command-line or via ~/.quiltrc,
etc., these are ignored if there are already files in ./pc/ containing
the "original" values, even when no patches are currently applied.

This caught me out whilst manually generating patches for a package that
covers multiple architectures and after initial work on a single
architecture I decided to create patch sub-dirs for each architecture
along with separate series files.

With the layout:

debian/patches/arm/
debian/patches/aarch64/
...

I then created series files for each initially as

debian/patches/series.${architecture}

and was confused that doing:

$ QUILT_SERIES=series.aarch64
$ quilt push

still used debian/patches/series that listed only the 'arm' patches.

Then I tried adding:

$ QUILT_PATCHES=debian/patches/aarch64
$ quilt push

And it still used the original ./debian/patches/series

Initially I thought either /etc/quilt.quiltrc or ~/.quiltrc were causing
this but commenting out QUILT_PATCHES in ~/.quiltrc didn't change the
result.

Eventually I read the quilt source which tipped me off that when the
./pc/ directory contains .quilt_series .quilt_patches etc., the environment
variable over-rides are ignored entirely.

This is surprising and unexpected especially when no patches are
currently applied.

man-page says:

"Files in the .pc directory are automatically removed when they are no longer
 needed, so there is no need to clean up manually"

But this makes me wonder what state is required for that to be the case;
the tool should certainly make clear if it ignores *command-line*  environment
variable over-rides but preferrably should honour them.

-- System Information:
Debian Release: 12.6
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable-security-debug'), (500, 
'stable-security'), (500, 'stable-debug'), (500, 'proposed-updates-debug'), 
(500, 'stable'), (100, 'proposed-updates')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Shell: /bin/sh linked to /usr/bin/bash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages quilt depends on:
ii  bsdextrautils   2.38.1-5+deb12u1
ii  bsdmainutils    12.1.8
ii  bzip2           1.0.8-5+b1
ii  diffstat        1.65-1
ii  ed              1.19-1
ii  gettext         0.21-12
ii  patch           2.7.6-7
ii  perl            5.36.0-7+deb12u1
ii  sensible-utils  0.0.17+nmu1

Versions of packages quilt recommends:
ii  less  590-2.1~deb12u2

Versions of packages quilt suggests:
ii  exim4-daemon-light [mail-transport-agent]  4.96-15+deb12u5
ii  graphviz                                   2.42.2-7+b3
pn  procmail                                   <none>

-- no debconf information

Reply via email to