lava_android_test.egg-info/SOURCES.txt does not mention setup.cfg

Bug #988710 reported by Marcin Juszkiewicz on 2012-04-26
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LAVA Android Test
Fix Released
Michael Hudson-Doyle

Bug Description

09:17 hrw@puchatek:lava-android-test-0.3$ LC_ALL=C debuild -S
 dpkg-buildpackage -rfakeroot -d -us -uc -S
dpkg-buildpackage: export CFLAGS from dpkg-buildflags (origin: vendor): -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security
dpkg-buildpackage: export CPPFLAGS from dpkg-buildflags (origin: vendor): -D_FORTIFY_SOURCE=2
dpkg-buildpackage: export CXXFLAGS from dpkg-buildflags (origin: vendor): -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security
dpkg-buildpackage: export FFLAGS from dpkg-buildflags (origin: vendor): -g -O2
dpkg-buildpackage: export LDFLAGS from dpkg-buildflags (origin: vendor): -Wl,-Bsymbolic-functions -Wl,-z,relro
dpkg-buildpackage: source package lava-android-test
dpkg-buildpackage: source version 0.3-0ubuntu1
dpkg-buildpackage: source changed by Marcin Juszkiewicz <email address hidden>
 dpkg-source --before-build lava-android-test-0.3
 fakeroot debian/rules clean
dh clean --with python2
running clean
'build/lib.linux-x86_64-2.7' does not exist -- can't clean it
'build/bdist.linux-x86_64' does not exist -- can't clean it
'build/scripts-2.7' does not exist -- can't clean it
 dpkg-source -b lava-android-test-0.3
dpkg-source: info: using source format `3.0 (quilt)'
dpkg-source: info: building lava-android-test using existing ./lava-android-test_0.3.orig.tar.gz
dpkg-source: info: local changes detected, the modified files are:
dpkg-source: error: aborting due to unexpected upstream changes, see /tmp/lava-android-test_0.3-0ubuntu1.diff.Cy1MNQ
dpkg-source: info: you can integrate the local changes with dpkg-source --commit
dpkg-buildpackage: error: dpkg-source -b lava-android-test-0.3 gave error exit status 2
debuild: fatal error at line 1350:
dpkg-buildpackage -rfakeroot -d -us -uc -S failed

Related branches

Marcin Juszkiewicz (hrw) wrote :

patch which I had to add to packaging

Michael Hudson-Doyle (mwhudson) wrote :

Hi, can you tell me how to reproduce this? The patch cannot be right unfortunately, the .egg-info directory is automatically generated by setuptools.

Marcin Juszkiewicz (hrw) wrote :

Add PPA to system (deb-src too).

apt-get build-dep lava-android-test
apt-get source lava-android-test
cd lava-android-test-0.3
dpkg-buildpackage -b
dpkg-buildpackage -S

Michael Hudson-Doyle (mwhudson) wrote :

Grar, this was a lot more confusing than I expected (partly because the file that was uploaded to Launchpad for the 2012.04 release is _not_ the same as the file on pypi).

What seems to happen is this:

 1. lp:lava-android-test does not contain a setup.cfg file
 2. when you run python sdist, this runs egg_info and so creates a SOURCES.txt that does not contain setup.cfg
 3. about the final thing sdist does before making the tarball is write a setup.cfg (this is a setuptools thing)
 4. when you unpack the tarball, debianize it and run dpkg-buildpackage, this runs egg_info, which now updates SOURCES.txt to contain setup.cfg
 5. when you run dpkg-buildpackage again dpkg-source complains because the tree has been modified.

I guess this is a setuptools bug really. But the workaround is easy: commit a setup.cfg file to lp:lava-android-test (most of our projects have one anyway to turn gpg signing on)

Yongqin Liu (liuyq0307) on 2012-06-07
Changed in lava-android-test:
importance: Undecided → Low
status: New → Fix Released
assignee: nobody → Michael Hudson-Doyle (mwhudson)
Fathi Boudra (fboudra) on 2012-06-13
Changed in lava-android-test:
milestone: none → 2012.05
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers