Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-23 Thread 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/c

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-23 Thread Jaromír Mikeš
2016-08-23 15:22 GMT+02:00 Daniel James : > I think the xwax package should be ready for final upload checks now. 1) I would add d/source/local-options file 2) Fix d/watch file to handle beta and rc releases correctly ardour's watch file can be helpful https://anonscm.debian.org/cgit/pkg-multime

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-23 Thread 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 admin

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-22 Thread Mattia Rizzolo
On Mon, Aug 22, 2016 at 08:39:29PM +0200, Jaromír Mikeš wrote: > I am enabling parallel build simply in d/rules file: > dh $@ --parallel That's the way it has to be done. (unless you're not using dh, but then you probably know your way) > I believe that number of processes is given by number

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-22 Thread Jaromír Mikeš
2016-08-22 17:09 GMT+02:00 Daniel James : Hi Daniel, >> 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 ok ... > I

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-22 Thread 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 an

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-22 Thread Jaromír Mikeš
2016-08-22 12:55 GMT+02:00 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

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-22 Thread 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 interest

Re: [SCM] xwax/master: Add bindnow hardening, disable pie for performance reasons

2016-08-19 Thread Bálint Réczey
Hi, 2016-08-16 15:46 GMT+02:00 : > The following commit has been merged in the master branch: > commit 48ef7795f583e619765180452bb7d3b6976e96a6 > Author: Daniel James > Date: Tue Aug 16 14:45:48 2016 +0100 > > Add bindnow hardening, disable pie for performance reasons > Do you have some g