Re: Bug#565860: still exists in 0.4.4-2

2010-01-24 Thread Alessio Treglia
On Sun, Jan 24, 2010 at 2:09 PM, Adrian Knoth  
wrote:
> That's what I do for jackd:
>
>   libasound2-dev [!kfreebsd-i386 !kfreebsd-amd64 !hurd-i386]
>

Thanks.


-- 
Alessio Treglia 
Ubuntu MOTU Developer | Homepage: http://www.alessiotreglia.com
0FEC 59A5 E18E E04F 6D40 593B 45D4 8C7C DCFC 3FD0

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-24 Thread Adrian Knoth
On Sun, Jan 24, 2010 at 01:58:08PM +0100, Alessio Treglia wrote:

> >> Couldn't you just change the Build-Depends from "libasound2-dev" to
> >> "libasound2-dev [linux-any]"?
> >>
> >
> > +1
> >
> > Seems the right way.
> ...but it doesn't work on my amd64 chroot, the build fails due to unmet deps.
> Should I set libasound2-dev [!kfreebsd-amd64 !kfreebsd-i386] instead
> of libasound2-dev [linux-any]?

That's what I do for jackd:

   libasound2-dev [!kfreebsd-i386 !kfreebsd-amd64 !hurd-i386]


HTH

-- 
mail: a...@thur.de  http://adi.thur.de  PGP/GPG: key via keyserver

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-24 Thread Alessio Treglia
On Wed, Jan 20, 2010 at 10:31 AM, Alessio Treglia  wrote:
> On Wed, Jan 20, 2010 at 10:26 AM, Fabian Greffrath
>  wrote:
>> Couldn't you just change the Build-Depends from "libasound2-dev" to
>> "libasound2-dev [linux-any]"?
>>
>
> +1
>
> Seems the right way.
>

...but it doesn't work on my amd64 chroot, the build fails due to unmet deps.
Should I set libasound2-dev [!kfreebsd-amd64 !kfreebsd-i386] instead
of libasound2-dev [linux-any]?



-- 
Alessio Treglia 
Ubuntu MOTU Developer | Homepage: http://www.alessiotreglia.com
0FEC 59A5 E18E E04F 6D40 593B 45D4 8C7C DCFC 3FD0

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Fabian Greffrath

Am 20.01.2010 11:23, schrieb Reinhard Tartler:

11:20  I used to prefer to ship patches against configure
   and Makefile.in's, but as time passes by, I changed
   my mind
11:20  debian's maintenance on autofoo tools has improved
   dramatically, IME


Funny coincidence. :)


Idempotency is preserved if the clean target removes all intermediate
files.


Yes, the maintenance effort of keeping up with the list of files to be 
removed should be about the same as refreshing a patch, I admit.



The price for this is the build dependency on autotools-dev and relying
on working autotools packages. I think we can afford both, but YMMV.


It's not that I don't trust the autofoo Debian packages. It's just 
that I think "why repeat the same steps again and again on every 
buildd if you already know the results and can apply them a priori?".


Cheers,
Fabian


--
Dipl.-Phys. Fabian Greffrath

Ruhr-Universität Bochum
Lehrstuhl für Energieanlagen und Energieprozesstechnik (LEAT)
Universitätsstr. 150, IB 3/134
D-44780 Bochum

Telefon: +49 (0)234 / 32-26334
Fax: +49 (0)234 / 32-14227
E-Mail:  greffr...@leat.ruhr-uni-bochum.de

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Reinhard Tartler
On Mi, Jan 20, 2010 at 10:53:06 (CET), Fabian Greffrath wrote:

> Am 19.01.2010 18:33, schrieb Adrian Knoth:
>> The patch was against configure.ac. I don't know if the minimal
>> debian/rules is sufficient to re-generate configure after applying this
>> patch.
>> If need be, add a line like "autoreconf -f" somewhere before calling
>> configure.
>
> Personally, instead of running autoreconf (or similar) during build, I'd
> prefer a patch that contains all the changes to configure caused by the
> changes to configure.ac.

11:20  I used to prefer to ship patches against configure
  and Makefile.in's, but as time passes by, I changed
  my mind
11:20  debian's maintenance on autofoo tools has improved
  dramatically, IME

> This way the package stays idempotent, i.e. it makes no difference if
> you build it once or twice from the same source and after running
> "debian/rules clean" everything is back in its initial state.

Idempotency is preserved if the clean target removes all intermediate
files.

The price for this is the build dependency on autotools-dev and relying
on working autotools packages. I think we can afford both, but YMMV.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Fabian Greffrath

Am 19.01.2010 18:33, schrieb Adrian Knoth:

The patch was against configure.ac. I don't know if the minimal
debian/rules is sufficient to re-generate configure after applying this
patch.
If need be, add a line like "autoreconf -f" somewhere before calling
configure.


Personally, instead of running autoreconf (or similar) during build, 
I'd prefer a patch that contains all the changes to configure caused 
by the changes to configure.ac. This way the package stays idempotent, 
i.e. it makes no difference if you build it once or twice from the 
same source and after running "debian/rules clean" everything is back 
in its initial state.


Fabian

--
Dipl.-Phys. Fabian Greffrath

Ruhr-Universität Bochum
Lehrstuhl für Energieanlagen und Energieprozesstechnik (LEAT)
Universitätsstr. 150, IB 3/134
D-44780 Bochum

Telefon: +49 (0)234 / 32-26334
Fax: +49 (0)234 / 32-14227
E-Mail:  greffr...@leat.ruhr-uni-bochum.de

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Alessio Treglia
On Wed, Jan 20, 2010 at 10:26 AM, Fabian Greffrath
 wrote:
> Couldn't you just change the Build-Depends from "libasound2-dev" to
> "libasound2-dev [linux-any]"?
>

+1

Seems the right way.

-- 
Alessio Treglia 
Ubuntu MOTU Developer | Homepage: http://www.alessiotreglia.com
0FEC 59A5 E18E E04F 6D40 593B 45D4 8C7C DCFC 3FD0

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Reinhard Tartler
On Mi, Jan 20, 2010 at 10:12:54 (CET), Adrian Knoth wrote:

> On Tue, Jan 19, 2010 at 09:02:21PM +0100, Alessio Treglia wrote:
>
> Hi!
>
>> I've already attached the correct patch (and pushed git repository).
>
> I've cloned the repository, and it works nicely on sparc.
>
> Looks like it's safe to upload.
>
> Just a last remark: I've seen build-errors for kfreebsd. Though I
> haven't looked closer, it could be the missing alsa-seq interface.
> (freebsd kernels don't have ALSA).
>
> You might need to disable the alsa-seq support on these platforms, but
> it's probably OK to wait for the FTBFS, first. ;)

I'd rather disable alsa-seq right now to avoid unnecessary build cycles
on slower buildds. Your explanation makes pretty much sense.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Fabian Greffrath

Am 20.01.2010 10:12, schrieb Adrian Knoth:

Just a last remark: I've seen build-errors for kfreebsd. Though I
haven't looked closer, it could be the missing alsa-seq interface.
(freebsd kernels don't have ALSA).

You might need to disable the alsa-seq support on these platforms, but
it's probably OK to wait for the FTBFS, first. ;)


Couldn't you just change the Build-Depends from "libasound2-dev" to 
"libasound2-dev [linux-any]"?



--
Dipl.-Phys. Fabian Greffrath

Ruhr-Universität Bochum
Lehrstuhl für Energieanlagen und Energieprozesstechnik (LEAT)
Universitätsstr. 150, IB 3/134
D-44780 Bochum

Telefon: +49 (0)234 / 32-26334
Fax: +49 (0)234 / 32-14227
E-Mail:  greffr...@leat.ruhr-uni-bochum.de

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-20 Thread Adrian Knoth
On Tue, Jan 19, 2010 at 09:02:21PM +0100, Alessio Treglia wrote:

Hi!

> I've already attached the correct patch (and pushed git repository).

I've cloned the repository, and it works nicely on sparc.

Looks like it's safe to upload.

Just a last remark: I've seen build-errors for kfreebsd. Though I
haven't looked closer, it could be the missing alsa-seq interface.
(freebsd kernels don't have ALSA).

You might need to disable the alsa-seq support on these platforms, but
it's probably OK to wait for the FTBFS, first. ;)


HTH

-- 
mail: a...@thur.de  http://adi.thur.de  PGP/GPG: key via keyserver

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-19 Thread Alessio Treglia
On Tue, Jan 19, 2010 at 8:35 PM, Reinhard Tartler  wrote:
>
> I think you might want to call autoreconf as 'autoreconf -i -f'.
>

I've already attached the correct patch (and pushed git repository).
Now that bug should be fixed.


-- 
Alessio Treglia 
Ubuntu MOTU Developer | Homepage: http://www.alessiotreglia.com
0FEC 59A5 E18E E04F 6D40 593B 45D4 8C7C DCFC 3FD0

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers


Re: Bug#565860: still exists in 0.4.4-2

2010-01-19 Thread Reinhard Tartler
On Di, Jan 19, 2010 at 19:48:51 (CET), Alessio Treglia wrote:

> --- a/debian/rules
> +++ b/debian/rules
> @@ -1,4 +1,8 @@
>  #!/usr/bin/make -f
>  
> +override_dh_auto_configure:
> + autoreconf
> + dh_auto_configure -f
> +
>  %:

I think you might want to call autoreconf as 'autoreconf -i -f'.


-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers