RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups
Dear mentors, I am looking for a sponsor for my package "ulatencyd". * Package name: ulatencyd Version : 0.4.5-1 Upstream Author : Daniel Poelzleithner * URL : https://github.com/poelzi/ulatencyd * License : GPL-3+ Section : utils It builds these binary packages: ulatencyd - Daemon to minimize latency on a linux system using cgroups The package appears to be lintian clean. The upload would fix these bugs: 610495 (ITP) My motivation for maintaining this package is: I've been using this software for a while now, and it proved to work very well. All the compatibility problems with Debian (mainly caused by missing Linux feature) have been resolved directly upstream (I've been in touch with the author for a while) and it now works with Debian's Linux packages nicely. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/u/ulatencyd - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.5-1.dsc Or on git.debian.org: - URL: http://git.debian.org/?p=collab-maint/ulatencyd.git - git clone git://git.debian.org/collab-maint/ulatencyd.git I would be glad if someone uploaded this package for me. Kind regards Alessandro Ghedini -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/4d5408a3.ca7a0e0a.4c10.0...@mx.google.com
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups
Alessandro Ghedini wrote: > Dear mentors, > > I am looking for a sponsor for my package "ulatencyd". Hi Alessandro, I am willing to review and sponsor your package, but don't have time right now for the actual review. I will make sure to send an answer to you during next week. Cheers, OdyX -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/ij96jr$tep$1...@dough.gmane.org
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups
On Sun, Feb 13, 2011 at 07:06:50PM +0100, Didier 'OdyX' Raboud wrote:
> Hi Alessandro,
Hello
> I am willing to review and sponsor your package, but don't have time right
> now for the actual review. I will make sure to send an answer to you during
> next week.
Thank you for your interest. Please note that I've just updated the package
to the latest upstream version.
The new package can be found on mentors.debian.net:
- dget
http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.6-1.dsc
And on git.debian.org (same links).
Take the time you need for the review :)
Cheers
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110213191321.GA28386@PC-Ale
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups
On Sun, Feb 13, 2011 at 07:06:50PM +0100, Didier 'OdyX' Raboud wrote:
> Hi Alessandro,
>
> I am willing to review and sponsor your package, but don't have time right
> now for the actual review. I will make sure to send an answer to you during
> next week.
Any news?
Thanks
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110221105602.GA28854@PC-Ale
RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
Dear mentors, I am looking for a sponsor for my package "ulatencyd". * Package name: ulatencyd Version : 0.4.6-1 Upstream Author : Daniel Poelzleithner * URL : https://github.com/poelzi/ulatencyd * License : GPL-3+ Section : utils It builds these binary packages: ulatencyd - Daemon to minimize latency on a linux system using cgroups The package appears to be lintian clean. The upload would fix these bugs: 610495 (ITP) My motivation for maintaining this package is: I've been using this software for a while now, and it proved to work very well. All the compatibility problems with Debian (mainly caused by missing Linux feature) have been resolved directly upstream (I've been in touch with the author for a while) and it now works with Debian's Linux packages nicely. The package can be found on mentors.debian.net: - URL: http://mentors.debian.net/debian/pool/main/u/ulatencyd - Source repository: deb-src http://mentors.debian.net/debian unstable main contrib non-free - dget http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.6-1.dsc Or on git.debian.org: - URL: http://git.debian.org/?p=collab-maint/ulatencyd.git - git clone git://git.debian.org/collab-maint/ulatencyd.git I would be glad if someone uploaded this package for me. Kind regards Alessandro Ghedini -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/20110227171926.GA16294@PC-Ale
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
Alessandro Ghedini wrote: > > Dear mentors, > > I am looking for a sponsor for my package "ulatencyd". Hi Alessandro, here is my (promised) review, with some delay; please forgive me for that; life took over… > ulatencyd - Daemon to minimize latency on a linux system using cgroups > > The package appears to be lintian clean. Confirmed. > The upload would fix these bugs: 610495 (ITP) Now some questions: * Why don't you ship the systemd service file? With systemd around the corner, you will certainly end up adding it in the future. And why are you stripping it away with a patch (where you could dh_auto_install to debian/tmp and have a "ulatencyd.install" file to opt files _in_) ? I would just correct the path in this install file and be done with it. * Your debian/init.d isn't named correctly (IMHO). man dh_installinit tells us that it should be named debian/ulatency.init (or debian/init, but I very much prefer being explicit). As for the names, it's the same for logrotate, manpages and docs (but don't worry, it's mostly a matter of taste). * Deactivation of the tests: why do you disable the tests ? Build tests should be run and they should not fail (obviously…). You should either comment your debian/rules explaining the reasons or (preferably) convince upstream to patch (or patch yourself) the tests in order to be able to run within the buildd environment. * debian/gbp.conf should not be in the source package; having a debian/source/local-options to filter it out sounds nice. Otherwise it sounds nice. Cheers, OdyX -- To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org Archive: http://lists.debian.org/iklrjp$k1i$1...@dough.gmane.org
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
Hello Didier,
On Wed, Mar 02, 2011 at 05:35:04PM +0100, Didier 'OdyX' Raboud wrote:
> here is my (promised) review, with some delay; please forgive me for that;
> life took over…
No problem, and thanks for the review.
I've just uploaded the new upstream version (0.4.9) of the package, released
a couple of days ago. It fixes some issues (e.g. the wrong systemd service file
path) and it also adds some new nice features. The most notable is the addition
of a 'ulatency' python script, which is basically, a simple client for the
daemon.
Since it depends on python and some other python modules I decided
to add a new binary package for it (called 'ulatency') so that if someone
do not want the client, he is not force to install all its python
dependencies.
Hope it's not a problem for you to review the new version as well.
You can find the new package on mentors.d.n:
dget
http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.9-1.dsc
As well as on git:
git clone git://git.debian.org/collab-maint/ulatencyd.git
> Now some questions:
>
> * Why don't you ship the systemd service file? With systemd around the
> corner, you will certainly end up adding it in the future. And why are you
> stripping it away with a patch (where you could dh_auto_install to
> debian/tmp and have a "ulatencyd.install" file to opt files _in_) ? I would
> just correct the path in this install file and be done with it.
Indeed. The wrong path issue has been fixed upstream, so that now the
systemd service file gets installed properly. Regarding the patch, I simply
found adding a patch easier, but your solution is more elegant.
In the new version of the package there's another patch that makes a file
to be installed. In this case I chose a patch to report more easily the
problem upstream (I've already forwarded that patch, and it is being
merged).
> * Your debian/init.d isn't named correctly (IMHO). man dh_installinit tells
> us that it should be named debian/ulatency.init (or debian/init, but I very
> much prefer being explicit). As for the names, it's the same for logrotate,
> manpages and docs (but don't worry, it's mostly a matter of taste).
I've fixed the names for all the files. Since now two binary packages
are built, it's not a matter of taste anymore (particularly for the
*.manpages ones) :)
> * Deactivation of the tests: why do you disable the tests ? Build tests
> should be run and they should not fail (obviously…). You should either
> comment your debian/rules explaining the reasons or (preferably) convince
> upstream to patch (or patch yourself) the tests in order to be able to run
> within the buildd environment.
The tests need the ulatencyd daemon to be running, which needs its core
library properly installed under /usr/lib/... AFAIK this is not possible at
build time, unless build-depending circularly on the ulatencyd package
itself. I have now documented this in the rules file though.
> * debian/gbp.conf should not be in the source package; having a
> debian/source/local-options to filter it out sounds nice.
Fixed... I think. I renamed the debian/gbp.conf to .gbp.conf and added
the extend-diff-ignore local-option to ignore it. Is this what you
suggested or is there a better solution?
Thank you again for the help.
Cheers
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110303151003.GA2640@PC-Ale
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
On Thu, Mar 03, 2011 at 04:10:03PM +0100, Alessandro Ghedini wrote:
> Hello Didier,
>
> On Wed, Mar 02, 2011 at 05:35:04PM +0100, Didier 'OdyX' Raboud wrote:
> > here is my (promised) review, with some delay; please forgive me for that;
> > life took over…
>
> No problem, and thanks for the review.
>
> I've just uploaded the new upstream version (0.4.9) of the package, released
> a couple of days ago. It fixes some issues (e.g. the wrong systemd service
> file path) and it also adds some new nice features. The most notable is the
> addition of a 'ulatency' python script, which is basically, a simple client
> for the daemon.
> Since it depends on python and some other python modules I decided
> to add a new binary package for it (called 'ulatency') so that if someone
> do not want the client, he is not force to install all its python
> dependencies.
>
> Hope it's not a problem for you to review the new version as well.
>
> You can find the new package on mentors.d.n:
> dget
> http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.9-1.dsc
>
> As well as on git:
> git clone git://git.debian.org/collab-maint/ulatencyd.git
>
> > Now some questions:
> >
> > * Why don't you ship the systemd service file? With systemd around the
> > corner, you will certainly end up adding it in the future. And why are you
> > stripping it away with a patch (where you could dh_auto_install to
> > debian/tmp and have a "ulatencyd.install" file to opt files _in_) ? I would
> > just correct the path in this install file and be done with it.
>
> Indeed. The wrong path issue has been fixed upstream, so that now the
> systemd service file gets installed properly. Regarding the patch, I simply
> found adding a patch easier, but your solution is more elegant.
>
> In the new version of the package there's another patch that makes a file
> to be installed. In this case I chose a patch to report more easily the
> problem upstream (I've already forwarded that patch, and it is being
> merged).
>
> > * Your debian/init.d isn't named correctly (IMHO). man dh_installinit tells
> > us that it should be named debian/ulatency.init (or debian/init, but I very
> > much prefer being explicit). As for the names, it's the same for logrotate,
> > manpages and docs (but don't worry, it's mostly a matter of taste).
>
> I've fixed the names for all the files. Since now two binary packages
> are built, it's not a matter of taste anymore (particularly for the
> *.manpages ones) :)
>
> > * Deactivation of the tests: why do you disable the tests ? Build tests
> > should be run and they should not fail (obviously…). You should either
> > comment your debian/rules explaining the reasons or (preferably) convince
> > upstream to patch (or patch yourself) the tests in order to be able to run
> > within the buildd environment.
>
> The tests need the ulatencyd daemon to be running, which needs its core
> library properly installed under /usr/lib/... AFAIK this is not possible at
> build time, unless build-depending circularly on the ulatencyd package
> itself. I have now documented this in the rules file though.
>
> > * debian/gbp.conf should not be in the source package; having a
> > debian/source/local-options to filter it out sounds nice.
>
> Fixed... I think. I renamed the debian/gbp.conf to .gbp.conf and added
> the extend-diff-ignore local-option to ignore it. Is this what you
> suggested or is there a better solution?
Any news about this?
Cheers
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110309215050.ga29...@pc-ale.rete
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
On Sun, Feb 27, 2011 at 06:19:26PM +0100, Alessandro Ghedini wrote:
> Dear mentors,
>
> I am looking for a sponsor for my package "ulatencyd".
>
> * Package name: ulatencyd
> Version : 0.4.6-1
> Upstream Author : Daniel Poelzleithner
> * URL : https://github.com/poelzi/ulatencyd
> * License : GPL-3+
> Section : utils
>
> It builds these binary packages:
> ulatencyd - Daemon to minimize latency on a linux system using cgroups
>
> The package appears to be lintian clean.
>
> The upload would fix these bugs: 610495 (ITP)
>
> My motivation for maintaining this package is: I've been using this
> software for a while now, and it proved to work very well. All the
> compatibility problems with Debian (mainly caused by missing Linux feature)
> have been resolved directly upstream (I've been in touch with the author
> for a while) and it now works with Debian's Linux packages nicely.
Ping?
The new version of the package (0.4.10) is on mentors.d.n:
- URL: http://mentors.debian.net/debian/pool/main/u/ulatencyd
- Source repository: deb-src http://mentors.debian.net/debian unstable main
- dget
http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.10-1.dsc
Cheers
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110313194545.ga2...@pc-ale.rete
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
On Sunday 13 March 2011 20:45:46 Alessandro Ghedini wrote: > Ping? > > The new version of the package (0.4.10) is on mentors.d.n: > - URL: http://mentors.debian.net/debian/pool/main/u/ulatencyd > - Source repository: deb-src http://mentors.debian.net/debian unstable main > - dget > http://mentors.debian.net/debian/pool/main/u/ulatencyd/ulatencyd_0.4.10-1. > dsc > > Cheers Hi Alessandro, sorry for my big delays, I didn't thought copyright review would be that boring… :-) Anyway, as I found no more comments to your package, I just uploaded it to the NEW queue, where it will sit in some moments. Regards, OdyX N.B. For future uploads, you can contact me directly (without CC'ing - mentors), I'd be glad to sponsor future uploads for you. signature.asc Description: This is a digitally signed message part.
Re: RFS: ulatencyd - Daemon to minimize latency on a linux system using cgroups (2nd try)
Hi Didier,
On Tue, Mar 15, 2011 at 04:47:55PM +0100, Didier Raboud wrote:
> sorry for my big delays, I didn't thought copyright review would be that
> boring… :-)
No problem, I understand what you mean :)
> Anyway, as I found no more comments to your package, I just uploaded it to
> the
> NEW queue, where it will sit in some moments.
Thank you very much.
> N.B. For future uploads, you can contact me directly (without CC'ing -
> mentors), I'd be glad to sponsor future uploads for you.
Sure.
Cheers
--
perl -E'$_=q;$/= @{[@_]};and s;\S+;;eg;say~~reverse'
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110315181352.ga5...@pc-ale.rete
