Re: [SCM] ecasound/master: Enable Large File support

2011-02-09 Thread Joel Roth 
On Wed, Feb 09, 2011 at 10:36:18PM +, ghedo-gu...@users.alioth.debian.org 
wrote:
> The following commit has been merged in the master branch:
> commit 719ef87d9794d6a3f375efa0d5b269fcd2325044
> Author: Alessandro Ghedini 
> Date:   Wed Feb 9 22:32:50 2011 +0100
> 
> Enable Large File support
> 
> diff --git a/debian/rules b/debian/rules
> index 20bebcf..372d2ce 100755
> --- a/debian/rules
> +++ b/debian/rules
> @@ -3,6 +3,9 @@
>  %:
>   dh $@ --with autotools-dev
>  
> +override_dh_auto_configure:
> + dh_auto_configure -- --enable-largefile

We don't need to enable this option: it is default behavior.

-
commit af792796fe9c3f87690b39a9b531d5ef1b53c0fb
Author: Kai Vehmanen 
Date:   Sun Feb 1 23:18:05 2009 +0200

Changed largefile support to be enabled by default if supported by the 
system.
-

>  override_dh_auto_clean:
>   mv ecasound.spec ecasound.spec.orig
>   dh_auto_clean
> 
> -- 
> ecasound packaging
> 
> ___
> pkg-multimedia-commits mailing list
> pkg-multimedia-comm...@lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-commits

-- 
Joel Roth

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

Re: [SCM] ecasound/master: Enable Large File support

2011-02-10 Thread Alessandro Ghedini 
On Wed, Feb 09, 2011 at 03:09:10PM -1000, Joel Roth wrote:
> We don't need to enable this option: it is default behavior.

I thought so, but I noticed that on my system it gets disabled, after 
running dh_auto_configure I see:

   Largefile support:  no

It may be a problem with my configuration (I didn't investigate any 
further), but explicitly enabling that option is not a big deal.

Cheers

-- 
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'

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

Re: [SCM] ecasound/master: Enable Large File support

2011-02-10 Thread IOhannes m zmoelnig 
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 2011-02-10 11:17, Alessandro Ghedini wrote:
> On Wed, Feb 09, 2011 at 03:09:10PM -1000, Joel Roth wrote:
>> We don't need to enable this option: it is default behavior.
> 
> I thought so, but I noticed that on my system it gets disabled, after 
> running dh_auto_configure I see:
> 
>Largefile support:  no
> 

as confirmed on the ecasound mailinglist, configure wrongly says "no" if
no special action has to be taken to enable large file support (it
doesn't really check whether it really won't support large files)

fgamsdr
IOhannes
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk1TvaQACgkQkX2Xpv6ydvTprwCdFdZ3fmbzt8olMO/Nk0VIGHZf
SP8AoORzshmvXIS7O+0qvF8cvOtE9TBl
=GitM
-END PGP SIGNATURE-



smime.p7s
Description: S/MIME Cryptographic Signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Re: [SCM] ecasound/master: Enable Large File support

2011-02-10 Thread Joel Roth 
On Thu, Feb 10, 2011 at 11:17:10AM +0100, Alessandro Ghedini wrote:
> On Wed, Feb 09, 2011 at 03:09:10PM -1000, Joel Roth wrote:
> > We don't need to enable this option: it is default behavior.
> 
> I thought so, but I noticed that on my system it gets disabled, after 
> running dh_auto_configure I see:
> 
>Largefile support:  no
> 
> It may be a problem with my configuration (I didn't investigate any 
> further), but explicitly enabling that option is not a big deal.

You're right, it's not a big deal. FWIW,
when I asked on the Ecasound list, Kai Vehmanen replied that the
option correctly means largefile support for 32bit system.
To avoid future confusion, ideally configure should be patched.


> When I run ./configure under Debian sid, the script   
>
> fails to find largefile support.  
>
>   
>
>checking for largefile support (>2GB files)... 
>
>checking for special C compiler options needed for large files... no   
>
>checking for _FILE_OFFSET_BITS value needed for large files... no  
>

   
Is this on a 64bit system perhaps? I believe on 64bit systems,  
   
_FILE_OFFSET_BITS has not effect, and then the above checks should be   
   
correct.
   

   
So if this is indeed the case, then the things are working as they  
   
should, but ecasound's configure summary is just misleading (it should  
   
say "largefile for 32bit systems...: yes/no".  
-

Best,

Joel

> 
> Cheers
> 
> -- 
> perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
> 
> ___
> pkg-multimedia-maintainers mailing list
> pkg-multimedia-maintainers@lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

-- 
Joel Roth

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