On Tue, Oct 09, 2018 at 11:44:34PM +0100, Ian Jackson wrote:
> I think this diff will fix it.  You can apply it directly with patch
> to dgit in your /usr/bin if you like.

Right, with your patch I see it's not crashing anymore.

However I also see that:
 * without --include-dirty, it's completely ignoring the added
   d/source/format file
 * with --iclude-dirty is giving up with
    error: tree contains uncommitted files and --clean=check specified
 * without -wc and with --include-dirty, it tries to its quilt
   linearisation, but it fails with:

mattia@warren ~/devel/debian/QA/mason/mason (git)-[remotes/dgit/dgit/sid] % 
dgit --include-dirty  build-source
dpkg-buildpackage: info: source package mason
dpkg-buildpackage: info: source version 1.0.0-12.4
dpkg-buildpackage: info: source distribution unstable
dpkg-buildpackage: info: source changed by Mattia Rizzolo <mat...@debian.org>
 fakeroot debian/rules clean
dh_testdir
dh_testroot
rm -f build-stamp install-stamp
debconf-updatepo
# Add here commands to clean up after the build process.
#-/usr/bin/make clean
dh_clean
dh_clean: Compatibility levels before 9 are deprecated (level 5 in use)
Format `3.0 (quilt)', need to check/update patch stack
examining quilt state (multiple patches, linear mode)
gzip: warning: GZIP environment variable is deprecated; use an alias or script
dgit: base trees orig=5df2f64c4d67150c7768 o+d/p=5df2f64c4d67150c7768
dgit: quilt differences: src:  ## orig ==     gitignores:  == orig ==
dgit: quilt differences:      HEAD ## o+d/p               HEAD == o+d/p
starting quiltify (multiple patches, linear mode)
quiltify linearisation planning successful, executing...
dpkg-source: warning: --include-removal is not a valid option for 
Dpkg::Source::Package::V1
dpkg-source: info: 'dpkg-source --commit' is not supported by the source format 
'1.0'
nothing quilty to commit, ok.
dpkg-source: info: using source format '3.0 (quilt)'
dpkg-source: info: building mason using existing ./mason_1.0.0.orig.tar.gz
dpkg-source: info: local changes detected, the modified files are:
 mason/Makefile
 mason/baserules.sample
 mason/firewall
 mason/mason
 mason/mason-decide
 mason/mason-gui-text
 mason/mason-gui-text.1
 mason/mason.1
 mason/masonlib
 mason/masonrc
 mason/samlib
 mason/ttt
dpkg-source: info: you can integrate the local changes with dpkg-source --commit
dpkg-source: error: aborting due to unexpected upstream changes, see 
/tmp/mason_1.0.0-12.4.diff.EP2mGJ
dgit: failed command: dpkg-source '-i(?:^|/)'\\'.git(?:/|$)' -I.git -b -- mason

dgit: error: subprocess failed with error exit status 2


I'm not sure if I'm doing something in a way I'm not supposed to do
here.
Note that I already worked around this bug yesterday by manually calling
dpkg-source --commit and git commit the resulting quilt patch, at which
point dgit goes back behaving the way I know it.

> I will make a release with the fix RSN.

I don't think there is a need to rush out a release just for this :)

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature

Reply via email to