Bug#535374: deejayd: bashism in debian/rules

2009-07-03 Thread Alexandre Rossi
 making it lintian -IE --pedantic clean (if you disagree with any of
 the tags then say why and we'll discuss it)
[...]
 lintian -IE --pedantic clean: is at least lintian clean, will check
 for -IE and --pedantic

$ lintian -IE --pedantic deejayd_0.8.2-1.dsc
I: deejayd source: build-depends-without-arch-dep debhelper
I: deejayd source: build-depends-without-arch-dep python

But both packages are needed to run the clean target, so I guess the
package is okay regarding this.

Alex



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#535374: deejayd: bashism in debian/rules

2009-07-02 Thread Alexandre Rossi
Hi,

 This is fixed in a version currently awaiting a sponsor at
 debian.mentors.net . As I do not have a sponsor for this package
 anymore, the fix may take time to reach the archive.

 I haven't seen you send a RFS to -mentors.

The current version should have been sponsored but my former sponsor
seems to busy with other things. I didn't want to bother him again.

 I may eventually sponsor it, but you should send a sponsorship request
 to -mentors first, and make some changes such as checking it against policy
 3.8.2, making it lintian -IE --pedantic clean (if you disagree with any of
 the tags then say why and we'll discuss it), and probably moving the direct
 changes in .diff.gz to a saner patch management system such as quilt or
 dpatch.

RFS: coming.
Policy 3.8.2: done in trunk.
lintian -IE --pedantic clean: is at least lintian clean, will check
for -IE and --pedantic
saner patch management: not an issue for the version in mentors, as
there are no changes outside the debian directory. Will fix whenever
that is not the case anymore. Anyway, individual patches are available
in the upstream VCS.

Thanks for your comments, and I'll send a RFS as soon as I've
addressed them, which should be this Week-End.

Alex



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#535374: deejayd: bashism in debian/rules

2009-07-01 Thread Alexandre Rossi
Hi,

 possible bashism in deejayd/rules line 42 (brace expansion):
         rm -f $(CURDIR)/man/*.{1,5}

This is fixed in a version currently awaiting a sponsor at
debian.mentors.net . As I do not have a sponsor for this package
anymore, the fix may take time to reach the archive.

I could also prepare an upload containing only the trivial fix for
this bug if this is of some use. Please let me know if so.

Alex



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#535374: deejayd: bashism in debian/rules

2009-07-01 Thread Raphael Geissert
Hi,

On Thursday 02 July 2009 00:09:29 Alexandre Rossi wrote:
 Hi,

  possible bashism in deejayd/rules line 42 (brace expansion):
          rm -f $(CURDIR)/man/*.{1,5}

 This is fixed in a version currently awaiting a sponsor at
 debian.mentors.net . As I do not have a sponsor for this package
 anymore, the fix may take time to reach the archive.

I haven't seen you send a RFS to -mentors.
I may eventually sponsor it, but you should send a sponsorship request 
to -mentors first, and make some changes such as checking it against policy 
3.8.2, making it lintian -IE --pedantic clean (if you disagree with any of 
the tags then say why and we'll discuss it), and probably moving the direct 
changes in .diff.gz to a saner patch management system such as quilt or 
dpatch.

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net


signature.asc
Description: This is a digitally signed message part.