Re: Help offered with xwax package

[Daniel James]

Hi James,

> Please mention that you added yourself to
> the list of uploaders in the changelog, then run dch -r, and then I'll
> upload it for you.

Thanks! That's done:
https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=19a0c911580a23b3aeefd446408cac3695f00516

Cheers!

Daniel

___
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: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

[Daniel James]

Hi Jaromír,

> 1) I would add d/source/local-options file

James Cowgill advised this was not required, I have no clue either way :-)

> 2) Fix d/watch file to handle beta and rc releases correctly
> ardour's watch file can be helpful

Thanks, this format works for me:

https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=d4a93d7032a31890e50357210c5977a096c48667

uscan --no-download --verbose
-- Scanning for watchfiles in .
-- Found watchfile in ./debian
-- In debian/watch, processing watchfile line:

opts=filenamemangle=s/.+\/v?(\d\S*)\.tar\.gz/xwax-$1\.tar\.gz/,uversionmangle=s/-beta/~beta/,dversionmangle=s/~dfsg.*//
  https://github.com/xwax/xwax/tags .*/v?(\d\S*)\.tar\.gz
-- Found the following matching hrefs:
 /xwax/xwax/archive/1.6.tar.gz (1.6)
 /xwax/xwax/archive/1.6-beta2.tar.gz (1.6~beta2)
 /xwax/xwax/archive/1.6-beta1.tar.gz (1.6~beta1)
 /xwax/xwax/archive/1.5.tar.gz (1.5)
 /xwax/xwax/archive/1.5-beta6.tar.gz (1.5~beta6)
 /xwax/xwax/archive/1.5-beta5.tar.gz (1.5~beta5)
 /xwax/xwax/archive/1.5-beta4.tar.gz (1.5~beta4)
 /xwax/xwax/archive/1.5-beta3.tar.gz (1.5~beta3)
 /xwax/xwax/archive/1.5-beta2.tar.gz (1.5~beta2)
 /xwax/xwax/archive/1.5-beta1.tar.gz (1.5~beta1)
Newest version on remote site is 1.6, local version is 1.6
 => Package is up to date
-- Scan finished

Cheers!

Daniel

___
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: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

[Daniel James]

Hi Mattia, hi Jaromír,

>> I am enabling parallel build simply in d/rules file:
>> dh $@ --parallel

Thanks, I've done that in:

https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=289ae2aec2cf2abbb2f8b39231b5a1a2cb91895a

> The level of parallelism is decided by the buildd administrator by
> exporting DEB_BUILD_OPTIONS=parallel=n, with n to its likings.

Ah, I misunderstood, I thought this was set in debian/rules for each
package, which made no sense to me if we don't know the build server
specification.

> n doesn't only depend on the number of cores.  The total number of RAM
> has to be considered too, as well whether there are going to be any
> other build ongoing on the same system at the same time.

Good point, thanks again for the explanation :-)

I think the xwax package should be ready for final upload checks now.

Cheers!

Daniel

___
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: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

[Daniel James]

Hi Jaromír,

> I didn't noticed any
> performance degradation.

Thanks, that's good to know. I've enabled all hardening options in
https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=0fb08b94d5a4a66ab44c580c8b16c1197c8be510

> And also no user complained.

That may not be proof of anything, especially if the user base for a
package is small.

I am unsure about how to enable parallel build, or if it is supported.
The doc https://www.debian.org/doc/debian-policy/ch-source.htm says to use:

DPKG_BUILD_OPTIONS=parallel=n

and in the note 27:

Packages built with make can often implement this by passing the -jn
option to make

...but doesn't suggest a reasonable value for n on Debian's build
servers. On a dual-core machine I could not find any obvious difference
when using a value of 2 for these parameters.

Cheers!

Daniel

___
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: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

[Daniel James]

Hi Bálint,

> Do you have some good tests/results for measuring the performance differences?

This is tricky, it would require benchmarking the app with various
potential configurations. That's why I'm wondering about enabling an
option which might have an impact I don't know about.

> I'm interested because I'm working on enabling those archive-wide.

I'd suggest that real-time audio applications are a special category,
because a performance degradation might make the program unusable.

Cheers!

Daniel

___
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: Help offered with xwax package

[Daniel James]

Hi Jaromír,

> 1. Bump of standards version
> 3. Use cgit for  Vcs-Browser would be probably better.
> 4. Is d/dir file needed ?

Thanks for the review! I have pushed the above changes in
https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=b26871620a6f5187d73907a684774477324f0045

> 2. Fix of hardening

As this is a real-time audio application, should hardening be applied if
it has a performance penalty? Does it offer any genuine security
advantage for a desktop application?

https://lintian.debian.org/tags/hardening-no-pie.html says "PIE has been
associated with noticeable performance overhead on i386. However, GCC-5
has implemented an optimization that can reduce the overhead
significantly."

As jessie provides gcc 4.9.2 and my aim is to provide a backport, it
seems like there could be a performance hit.

> 5. Is parallel build now default or should be enabled?

The debian/rules file is very customised, I don't see any parallel build
options used there at the moment.

> 6. I would add d/source/local-options file

Is this file still needed even when the upstream source is no longer
patched? The previous patch supporting avconv was now merged upstream.

Cheers!

Daniel

___
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: Help offered with xwax package

[Daniel James]

Hi Jaromír, hi all,

> Or maybe the best ... asking upstream if he could name his releases
> with ~ as - breaking our automatic system for watching new releases.

I talked to the upstream author, he had not intended beta releases to be
packaged by downstream distros.

Anyway, the patch from Debian has been merged upstream and a 1.6 final
release was made. I have imported these sources and removed the patch.

Please could someone from the pkg-multimedia team review the package for
release?

https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/

I have built it locally on jessie amd64 in a pbuilder-dist chroot and it
seems fine.

Thanks!

Daniel

___
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: Help offered with xwax package

[Daniel James]

Hi Jaromír, hi James,

> Or maybe the best ... asking upstream if he could name his releases
> with ~ as - breaking our automatic system for watching new releases.

I don't think that's very likely in this case, as the 1.6-beta2 tag was
made a while ago. It might be better to handle this discrepancy at the
Debian end.

I'd appreciate any help with getting this package uploaded correctly.

Cheers!

Daniel

___
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: Help offered with xwax package

[Daniel James]

Hi Jaromír,

>> looks like named orig source as 1.6-beta2-1 instead of 1.6~beta2-1
>> > (-/~) which cause problem when building with --pristine-tar

Yes, that was probably my fault, the upstream release name is 1.6-beta2.

> How did you get orig tarball? ... I guess that you download it
> manually because watch file showing 1.5 release as latest release.

I archived it from a local clone of https://github.com/xwax/xwax/ but on
http://xwax.org/download.html version 1.5 is shown as the latest.

The beta cycle is very long for xwax, so it makes sense that Debian
packaged the beta1 release already.

> Let's fix watch file first to new location of releases.

I've done that in:

https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/commit/?id=ee27698b3abfed1810e0a245d687d7f9e021bfa1

> Than $ uscan --debug --force-download should give you latest release
> automatically.

uscan works, but there is a slight discrepancy because of the tilde in
the Debian version:

$ uscan --report-status
Processing watchfile line for package xwax...
Newest version on remote site is 1.6-beta2, local version is 1.6~beta2
xwax: Newer version (1.6-beta2) available on remote site:
  https://github.com/xwax/xwax/archive/1.6-beta2.zip
  (local version is 1.6~beta2)

Cheers!

Daniel

___
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: [SCM] xwax/master: Fix some Lintian warnings by changing version number etc.

[Daniel James]

Hi Jaromír,

> Just hint urgency=medium is default now and "UNRELEASED" instead of
> "unstable" should be set before uploading.

Thanks for the tip, I've updated the changelog.

Cheers!

Daniel

___
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: [SCM] xwax/master: Fix some Lintian warnings by changing version number etc.

[Daniel James]

Hi James,

> Assuming you want to continue maintaining this package, you should
> revert these changelog changes and instead add yourself to the list of
> Uploaders.

Thanks for taking a look :-)

I've added myself to Uploaders and updated the changelog, as I found one
other issue in the README.debian file.

Cheers!

Daniel

___
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: Help offered with xwax package

[Daniel James]

Hi Felipe, hi Jaromír,

> Welcome on board ;)

Thanks! I have pushed some updates to
https://anonscm.debian.org/cgit/pkg-multimedia/xwax.git/ which are ready
for review. I did not push a debian/1.6_beta2* tag yet.

I have built the updated package in a clean jessie amd64 chroot using
pubuilder-dist, and it installed and worked fine on my jessie system.
Test .deb packages for amd64 and armhf are available here:
https://github.com/danielhjames/xwax-debs/tree/master/jessie

The upstream git repo still includes a file called .version in the root
directory but it is listed in .gitignore there, so when I created the
archive of the new upstream version I think this file was left out:

git archive 1.6-beta2|gzip > ../xwax-1.6~beta2-orig.tar.gz

I don't think this .version file is used by Debian in any way, so I'm
just mentioning it for the sake of completeness.

Cheers!

Daniel

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

Help offered with xwax package

[Daniel James]

Hi all,

I'd like to help update the xwax package in Debian to the latest
upstream 1.6-beta2, which Ubuntu already has. Then I would like to
backport this new version to jessie-backports, as jessie currently has
xwax version 1.5.

There appears to be a typo in the current patch for Debian
0001-use_either_ffmpeg_or_avconv.patch as follows:

which avconf -> which avconv

I attach an updated copy of the patch.

I am not a Debian Developer but I have done some packaging work in
collab-maint. If it would be possible to have commit access to
pkg-multimedia/xwax on alioth I would gladly push some other fixes
there, for lintian issues etc.

My alioth user name is dhj-guest.

Cheers!

Daniel
Description: Pick either avconv or ffmpeg.
Author: Alessio Treglia 
Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=722487
Forwarded: no
---
 import |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- xwax.orig/import
+++ xwax/import
@@ -13,6 +13,8 @@
 FILE="$1"
 RATE="$2"
 
+g_fallback_tool="$(which avconv || which ffmpeg)"
+
 case "$FILE" in
 
 *.cdaudio)
@@ -27,7 +29,7 @@ case "$FILE" in
 
 *)
 echo "Calling fallback decoder..." >&2
-exec ffmpeg -v 0 -i "$FILE" -f s16le -ar "$RATE" -
+exec "$g_fallback_tool" -v 0 -i "$FILE" -f s16le -ar "$RATE" -
 ;;
 
 esac
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Bug#779968: icecast2: Please package Icecast 2.4.1

[Daniel James]

This ticket should be for Icecast 2.4.2 now, due to the security issue
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=782120

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

Bug#736832: closed by Sebastian Ramacher sramac...@debian.org (Re: Bug#736832: Double dependencies for vlc-nox)

[Daniel James]

Hi Sebastian,

I don't mind this bug being closed if you think it isn't worth fixing.

Just for the record, this is not an artefact of packages.debian.org -
taking a look at the debian/control file for the binary vlc-nox package
in sid, you can see that...

1. dependencies are listed on libavcodec55 plus most or all of the
dependencies of libavcodec55, which is unnecessary, since depending on
libavcodec55 should be enough.

2. there is a Pre-Depends: dpkg (= 1.15.6~) which should no longer be
required, since all versions of dpkg currently available in Debian are
newer than this.

Cheers!

Daniel

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

Bug#736832: closed by Sebastian Ramacher sramac...@debian.org (Re: Bug#736832: Double dependencies for vlc-nox)

[Daniel James]

Thanks for the explanation!

Daniel

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

Bug#736832: Double dependencies for vlc-nox

[Daniel James]

Package: vlc-nox
Version: 2.1.2-2+b1
Severity: wishlist

Hello,

Looking at the very long list of dependencies for vlc-nox:

http://packages.debian.org/sid/vlc-nox

I noticed there is a dependency on libavcodec53 or libavcodec54. However
many of the same dependencies are listed for both libavcodec54 and
vlc-nox, such as:

 libavutil52 (= 6:9.1-1)
 libspeex1 (= 1.2~beta3-1)
 libtheora0 (= 1.0)
 libvorbis0a (= 1.1.2)
 libvorbisenc2 (= 1.1.2)
 libx264-133
 zlib1g (= 1:1.2.0.2)

The dependency on zlib1g is slightly higher for vlc-nox, but it's an old
version in both cases.

Also one dependency of vlc-nox is essential, dpkg, albeit on an old
version (= 1.15.6~) - is it necessary to list this any more?

It would be a bit easier to read the dependency list for vlc-nox if it
wasn't quite so long!

Thanks!

Daniel

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

Bug#715361: Please package Silan update

[Daniel James]

Package: silan
Version: 0.2.0+20121230.gitb1d80bcc8d-1
Severity: normal

Hello Debian Multimedia Maintainers,

Could the Silan package in sid be updated to upstream's 0.3.2 please?
There are many useful fixes:

http://rg42.org/gitweb/?p=silan.git;a=shortlog;h=refs/tags/v0.3.2

Airtime (http://www.sourcefabric.org/en/airtime/) requires a recent
version of Silan, so this update would help us package Airtime for Debian.

Thanks!

Daniel

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