Re: Bug#668505: dwarf fortress debian package

2013-09-05 Thread Russ Allbery
Russ Allbery writes: > Beren Minor writes: >> I can either: >> - exclude usr/lib/gemrb from dh_makeshlibs > I would do this. If the library is not a public API that should be used > by other projects, then having out it ouf /usr/lib is probably the right > decision (particularly if that's what

Re: Bug#668505: dwarf fortress debian package

2013-09-05 Thread Beren Minor
Hi, I have resolved the lintian warnings (or overrided them in case of false positives), pushed the changes to the pkg-games repository[1] and re-uploaded the package to mentors [2]. [1] h

Re: Bug#668505: dwarf fortress debian package

2013-09-05 Thread Russ Allbery
Beren Minor writes: > There is indeed one shared library in the package, located in > usr/lib/gemrb/. The library is only meant to be used by the game > executable, and that's why it is located here. The executable does not > need any help from ldconfig as it has its rpath pointing directly to >

Re: Bug#668505: dwarf fortress debian package

2013-09-05 Thread Beren Minor
Thanks for the explanations ! Things are clearer now. There is indeed one shared library in the package, located in usr/lib/gemrb/. The library is only meant to be used by the game executable, and that's why it is located here. The executable does not need any help from ldconfig as it has its rpat

Re: Bug#668505: dwarf fortress debian package

2013-09-04 Thread Russ Allbery
Beren Minor writes: > On Wed, Sep 4, 2013 at 12:49 AM, Russ Allbery wrote: >> Alternately (and in my opinion preferrably) just exclude the plugins >> directory from dh_makeshlibs: >> override_dh_makeshlibs: >> dh_makeshlibs -Xusr/lib/gemrb/plugins >> (This is really a (minor) u

Re: Bug#668505: dwarf fortress debian package

2013-09-03 Thread Beren Minor
On Wed, Sep 4, 2013 at 12:49 AM, Russ Allbery wrote: > > Alternately (and in my opinion preferrably) just exclude the plugins > directory from dh_makeshlibs: > > override_dh_makeshlibs: > dh_makeshlibs -Xusr/lib/gemrb/plugins > > (This is really a (minor) upstream bug, since, as pl

Re: Bug#668505: dwarf fortress debian package

2013-09-03 Thread Russ Allbery
Scott Howard writes: > 3) Lintian report: > I: gemrb: desktop-entry-lacks-keywords-entry > usr/share/applications/gemrb.desktop > I: libgemrb: hardening-no-fortify-functions > usr/lib/gemrb/plugins/2DAImporter.so > I: libgemrb: hardening-no-fortify-functions > usr/lib/gemrb/plugins/BAMImporter.

Bug#668505: dwarf fortress debian package

2013-09-03 Thread Beren Minor
On Tue, Sep 3, 2013 at 4:13 PM, Scott Howard wrote: > 1) Changelog is confusing, there are multiple entries claiming upload > to unstable that has never been uploaded. The ITP bug is closed very > low on the list. Also, this is labeled as 0.8.0-3, but this is the > first upload to Debian. For the

Bug#668505: dwarf fortress debian package

2013-09-03 Thread Scott Howard
On Tue, Sep 3, 2013 at 5:45 AM, Beren Minor wrote: > BTW, I also have a GemRB package that requires sponsorship. I've > recently joined the Debian Games Team to maintain it, but I have > little time to chase sponsors. So, if you have some time to spend and > want to have a look at it, it would be