Re: liblscp -packaging

2016-11-29 Thread Jaromír Mikeš
2016-11-29 23:12 GMT+01:00 James Cowgill :
> On 29/11/16 08:07, Jaromír Mikeš wrote:
>> 2016-11-23 23:50 GMT+01:00 Jaromír Mikeš :
>>> 2016-11-23 22:47 GMT+01:00 James Cowgill :
 On 16/11/16 08:02, Jaromír Mikeš wrote:
> 2016-11-15 23:42 GMT+01:00 James Cowgill :
>> On 15/11/16 12:28, Ondrej Novy wrote:
>>> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
 after building liblscp some files gets deleted
>>>
>>> i would use extend-diff-ignore in d/source/options, because it looks
>>> like "just cache". Maybe upstream should remove this files from release 
>>> too.
>>
>> I don't think that option is applicable here. dpkg-source by default
>> already ignores deleted files which are present in the orig tarball.
>> There's nothing needed to get dpkg-source to work.
>>
>> I'm not sure how you'd get git-buildpackage to ignore just this
>> directory though. Usually I just ignore stuff like this and 'git
>> checkout --' whenever I need to use gbp. Try and get upstream to remove
>> it in the next release.
>
> thank you!
> liblscp ready is for upload now.

 A few comments (and sorry I've been quite busy this past week):

 I don't understand why you have now decided to repack the source. Why do
 you need to remove autom4te.cache? The build works fine even with it.

 You could remove 01-makefiles.patch.

 Since this is a C library, you should add a symbols file.

 dh --with autoreconf is not needed with compat 10
>>
>> ping ;)
>
> Done, but please fix things so you don't need to repack the orig tarball
> before uploading this.

Fixed and uploaded. Thank you

mira

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

Re: liblscp -packaging

2016-11-29 Thread James Cowgill
On 29/11/16 08:07, Jaromír Mikeš wrote:
> 2016-11-23 23:50 GMT+01:00 Jaromír Mikeš :
>> 2016-11-23 22:47 GMT+01:00 James Cowgill :
>>> On 16/11/16 08:02, Jaromír Mikeš wrote:
 2016-11-15 23:42 GMT+01:00 James Cowgill :
> On 15/11/16 12:28, Ondrej Novy wrote:
>> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>>> after building liblscp some files gets deleted
>>
>> i would use extend-diff-ignore in d/source/options, because it looks
>> like "just cache". Maybe upstream should remove this files from release 
>> too.
>
> I don't think that option is applicable here. dpkg-source by default
> already ignores deleted files which are present in the orig tarball.
> There's nothing needed to get dpkg-source to work.
>
> I'm not sure how you'd get git-buildpackage to ignore just this
> directory though. Usually I just ignore stuff like this and 'git
> checkout --' whenever I need to use gbp. Try and get upstream to remove
> it in the next release.

 thank you!
 liblscp ready is for upload now.
>>>
>>> A few comments (and sorry I've been quite busy this past week):
>>>
>>> I don't understand why you have now decided to repack the source. Why do
>>> you need to remove autom4te.cache? The build works fine even with it.
>>>
>>> You could remove 01-makefiles.patch.
>>>
>>> Since this is a C library, you should add a symbols file.
>>>
>>> dh --with autoreconf is not needed with compat 10
> 
> ping ;)

Done, but please fix things so you don't need to repack the orig tarball
before uploading this.

James



signature.asc
Description: OpenPGP digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: liblscp -packaging

2016-11-29 Thread Jaromír Mikeš
2016-11-23 23:50 GMT+01:00 Jaromír Mikeš :
> 2016-11-23 22:47 GMT+01:00 James Cowgill :
>> Hi,
>>
>> On 16/11/16 08:02, Jaromír Mikeš wrote:
>>> 2016-11-15 23:42 GMT+01:00 James Cowgill :
 On 15/11/16 12:28, Ondrej Novy wrote:
> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>> after building liblscp some files gets deleted
>
> i would use extend-diff-ignore in d/source/options, because it looks
> like "just cache". Maybe upstream should remove this files from release 
> too.

 I don't think that option is applicable here. dpkg-source by default
 already ignores deleted files which are present in the orig tarball.
 There's nothing needed to get dpkg-source to work.

 I'm not sure how you'd get git-buildpackage to ignore just this
 directory though. Usually I just ignore stuff like this and 'git
 checkout --' whenever I need to use gbp. Try and get upstream to remove
 it in the next release.
>>>
>>> thank you!
>>> liblscp ready is for upload now.
>>
>> A few comments (and sorry I've been quite busy this past week):
>>
>> I don't understand why you have now decided to repack the source. Why do
>> you need to remove autom4te.cache? The build works fine even with it.
>>
>> You could remove 01-makefiles.patch.
>>
>> Since this is a C library, you should add a symbols file.
>>
>> dh --with autoreconf is not needed with compat 10

ping ;)

mira

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

Re: liblscp -packaging

2016-11-23 Thread Jaromír Mikeš
2016-11-23 22:47 GMT+01:00 James Cowgill :
> Hi,
>
> On 16/11/16 08:02, Jaromír Mikeš wrote:
>> 2016-11-15 23:42 GMT+01:00 James Cowgill :
>>> On 15/11/16 12:28, Ondrej Novy wrote:
 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
> after building liblscp some files gets deleted

 i would use extend-diff-ignore in d/source/options, because it looks
 like "just cache". Maybe upstream should remove this files from release 
 too.
>>>
>>> I don't think that option is applicable here. dpkg-source by default
>>> already ignores deleted files which are present in the orig tarball.
>>> There's nothing needed to get dpkg-source to work.
>>>
>>> I'm not sure how you'd get git-buildpackage to ignore just this
>>> directory though. Usually I just ignore stuff like this and 'git
>>> checkout --' whenever I need to use gbp. Try and get upstream to remove
>>> it in the next release.
>>
>> thank you!
>> liblscp ready is for upload now.
>
> A few comments (and sorry I've been quite busy this past week):
>
> I don't understand why you have now decided to repack the source. Why do
> you need to remove autom4te.cache? The build works fine even with it.
>
> You could remove 01-makefiles.patch.
>
> Since this is a C library, you should add a symbols file.
>
> dh --with autoreconf is not needed with compat 10

Done!

mira

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

Re: liblscp -packaging

2016-11-23 Thread Jaromír Mikeš
2016-11-23 23:19 GMT+01:00 James Cowgill :
> On 23/11/16 22:15, Jaromír Mikeš wrote:
>> 2016-11-23 22:47 GMT+01:00 James Cowgill :
>>> On 16/11/16 08:02, Jaromír Mikeš wrote:
 2016-11-15 23:42 GMT+01:00 James Cowgill :
> On 15/11/16 12:28, Ondrej Novy wrote:
>> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>>> after building liblscp some files gets deleted
>>
>> i would use extend-diff-ignore in d/source/options, because it looks
>> like "just cache". Maybe upstream should remove this files from release 
>> too.
>
> I don't think that option is applicable here. dpkg-source by default
> already ignores deleted files which are present in the orig tarball.
> There's nothing needed to get dpkg-source to work.
>
> I'm not sure how you'd get git-buildpackage to ignore just this
> directory though. Usually I just ignore stuff like this and 'git
> checkout --' whenever I need to use gbp. Try and get upstream to remove
> it in the next release.

 thank you!
 liblscp ready is for upload now.
>>>
>>> A few comments (and sorry I've been quite busy this past week):
>>>
>>> I don't understand why you have now decided to repack the source. Why do
>>> you need to remove autom4te.cache? The build works fine even with it.
>>
>> I have been in contact with upstream and asked him to remove this dir
>> in future release ...
>> The answer was ... something like ... "I am ok with this on opensuse
>> ... so I don't care"
>> Then I think is better for us to take care of this issue this way now
>> and in future.
>
> So why do you need to remove this directory in the first place?

This directory has been always deleted during a build and makes source "dirty"
I didn't found any better solution for this problem.

mira

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

Re: liblscp -packaging

2016-11-23 Thread James Cowgill
On 23/11/16 22:15, Jaromír Mikeš wrote:
> 2016-11-23 22:47 GMT+01:00 James Cowgill :
>> On 16/11/16 08:02, Jaromír Mikeš wrote:
>>> 2016-11-15 23:42 GMT+01:00 James Cowgill :
 On 15/11/16 12:28, Ondrej Novy wrote:
> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>> after building liblscp some files gets deleted
>
> i would use extend-diff-ignore in d/source/options, because it looks
> like "just cache". Maybe upstream should remove this files from release 
> too.

 I don't think that option is applicable here. dpkg-source by default
 already ignores deleted files which are present in the orig tarball.
 There's nothing needed to get dpkg-source to work.

 I'm not sure how you'd get git-buildpackage to ignore just this
 directory though. Usually I just ignore stuff like this and 'git
 checkout --' whenever I need to use gbp. Try and get upstream to remove
 it in the next release.
>>>
>>> thank you!
>>> liblscp ready is for upload now.
>>
>> A few comments (and sorry I've been quite busy this past week):
>>
>> I don't understand why you have now decided to repack the source. Why do
>> you need to remove autom4te.cache? The build works fine even with it.
> 
> I have been in contact with upstream and asked him to remove this dir
> in future release ...
> The answer was ... something like ... "I am ok with this on opensuse
> ... so I don't care"
> Then I think is better for us to take care of this issue this way now
> and in future.

So why do you need to remove this directory in the first place?

James



signature.asc
Description: OpenPGP digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: liblscp -packaging

2016-11-23 Thread Jaromír Mikeš
2016-11-23 22:47 GMT+01:00 James Cowgill :
> Hi,
>
> On 16/11/16 08:02, Jaromír Mikeš wrote:
>> 2016-11-15 23:42 GMT+01:00 James Cowgill :
>>> On 15/11/16 12:28, Ondrej Novy wrote:
 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
> after building liblscp some files gets deleted

 i would use extend-diff-ignore in d/source/options, because it looks
 like "just cache". Maybe upstream should remove this files from release 
 too.
>>>
>>> I don't think that option is applicable here. dpkg-source by default
>>> already ignores deleted files which are present in the orig tarball.
>>> There's nothing needed to get dpkg-source to work.
>>>
>>> I'm not sure how you'd get git-buildpackage to ignore just this
>>> directory though. Usually I just ignore stuff like this and 'git
>>> checkout --' whenever I need to use gbp. Try and get upstream to remove
>>> it in the next release.
>>
>> thank you!
>> liblscp ready is for upload now.
>
> A few comments (and sorry I've been quite busy this past week):
>
> I don't understand why you have now decided to repack the source. Why do
> you need to remove autom4te.cache? The build works fine even with it.

I have been in contact with upstream and asked him to remove this dir
in future release ...
The answer was ... something like ... "I am ok with this on opensuse
... so I don't care"
Then I think is better for us to take care of this issue this way now
and in future.

> You could remove 01-makefiles.patch.

Ok

> Since this is a C library, you should add a symbols file.

Ok I will

> dh --with autoreconf is not needed with compat 10

Ok ... Thanks

mira

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

Re: liblscp -packaging

2016-11-23 Thread James Cowgill
Hi,

On 16/11/16 08:02, Jaromír Mikeš wrote:
> 2016-11-15 23:42 GMT+01:00 James Cowgill :
>> On 15/11/16 12:28, Ondrej Novy wrote:
>>> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
 after building liblscp some files gets deleted
>>>
>>> i would use extend-diff-ignore in d/source/options, because it looks
>>> like "just cache". Maybe upstream should remove this files from release too.
>>
>> I don't think that option is applicable here. dpkg-source by default
>> already ignores deleted files which are present in the orig tarball.
>> There's nothing needed to get dpkg-source to work.
>>
>> I'm not sure how you'd get git-buildpackage to ignore just this
>> directory though. Usually I just ignore stuff like this and 'git
>> checkout --' whenever I need to use gbp. Try and get upstream to remove
>> it in the next release.
> 
> thank you!
> liblscp ready is for upload now.

A few comments (and sorry I've been quite busy this past week):

I don't understand why you have now decided to repack the source. Why do
you need to remove autom4te.cache? The build works fine even with it.

You could remove 01-makefiles.patch.

Since this is a C library, you should add a symbols file.

dh --with autoreconf is not needed with compat 10

Thanks,
James



signature.asc
Description: OpenPGP digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: liblscp -packaging

2016-11-16 Thread Jaromír Mikeš
2016-11-15 23:42 GMT+01:00 James Cowgill :
> Hi,
>
> On 15/11/16 12:28, Ondrej Novy wrote:
>> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>>> after building liblscp some files gets deleted
>>
>> i would use extend-diff-ignore in d/source/options, because it looks
>> like "just cache". Maybe upstream should remove this files from release too.
>
> I don't think that option is applicable here. dpkg-source by default
> already ignores deleted files which are present in the orig tarball.
> There's nothing needed to get dpkg-source to work.
>
> I'm not sure how you'd get git-buildpackage to ignore just this
> directory though. Usually I just ignore stuff like this and 'git
> checkout --' whenever I need to use gbp. Try and get upstream to remove
> it in the next release.

thank you!
liblscp ready is for upload now.

Can someone provide DM flag for me please?

best regards

mira

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

Re: liblscp -packaging

2016-11-15 Thread James Cowgill
Hi,

On 15/11/16 12:28, Ondrej Novy wrote:
> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>> after building liblscp some files gets deleted
> 
> i would use extend-diff-ignore in d/source/options, because it looks
> like "just cache". Maybe upstream should remove this files from release too.

I don't think that option is applicable here. dpkg-source by default
already ignores deleted files which are present in the orig tarball.
There's nothing needed to get dpkg-source to work.

I'm not sure how you'd get git-buildpackage to ignore just this
directory though. Usually I just ignore stuff like this and 'git
checkout --' whenever I need to use gbp. Try and get upstream to remove
it in the next release.

James



signature.asc
Description: OpenPGP digital signature
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Re: liblscp -packaging

2016-11-15 Thread Jaromír Mikeš
Hi,

2016-11-15 13:28 GMT+01:00 Ondrej Novy :
> 2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :
>>
>> after building liblscp some files gets deleted
>
> i would use extend-diff-ignore in d/source/options, because it looks like
> "just cache". Maybe upstream should remove this files from release too.

I added this line to d/source/options file:

extend-diff-ignore =
"(^|/)(autom4te\.cache/output\.0|autom4te\.cache/output\.1|autom4te\.cache/requests|autom4te\.cache/traces\.0|autom4te\.cache/traces\.1))$"

but it doesn't work :(

dpkg-source: info: using options from
liblscp/debian/source/local-options: --unapply-patches
--abort-on-upstream-changes
dpkg-source: info: using options from liblscp/debian/source/options:
--extend-diff-ignore=(^|/)(autom4te.cache/output\.0|autom4te.cache/output\.1|autom4te.cache/requests|autom4te.cache/traces\.0|autom4te.cache/traces\.1))$
dpkg-source: info: using source format '3.0 (quilt)'
dpkg-source: info: building liblscp using existing ./liblscp_0.5.8.orig.tar.gz
Unmatched ) in regex; marked by <-- HERE in
m/(?:^|/).*~$|(?:^|/)\.#.*$|(?:^|/)\..*\.sw.$|(?:^|/),,.*(?:$|/.*$)|(?:^|/)(?:DEADJOE|\.arch-inventory|\.(?:bzr|cvs|hg|git)ignore)$|(?:^|/)(?:CVS|RCS|\.deps|\{arch\}|\.arch-ids|\.svn|\.hg(?:tags|sigs)?|_darcs|\.git(?:attributes|modules|review)?|\.mailmap|\.shelf|_MTN|\.be|\.bzr(?:\.backup|tags)?)(?:$|/.*$)|(^|/)(autom4te.cache/output\.0|autom4te.cache/output\.1|autom4te.cache/requests|autom4te.cache/traces\.0|autom4te.cache/traces\.1))
<-- HERE $|(?:^|/)debian/source/local-.*$/ at
/usr/share/perl5/Dpkg/Source/Package/V3/Quilt.pm line 193.
dpkg-buildpackage: error: dpkg-source -b liblscp gave error exit status 2
gbp:error: 'git-pbuilder' failed: it exited with 2

mira

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

Re: liblscp -packaging

2016-11-15 Thread Ondrej Novy
Hi,

2016-11-15 11:47 GMT+01:00 Jaromír Mikeš :

> after building liblscp some files gets deleted
>

i would use extend-diff-ignore in d/source/options, because it looks like
"just cache". Maybe upstream should remove this files from release too.

-- 
S pozdravem/Best regards
 Ondřej Nový

Email: n...@ondrej.org
PGP: 3D98 3C52 EB85 980C 46A5  6090 3573 1255 9D1E 064B
Jabber: on...@njs.netlab.cz
ICQ: 115-674-713
Facebook: http://www.facebook.com/onovy
Tel/Cell: +420 777 963 207
Datová schránka: aypqr6c
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers