Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-13 Thread Jacob Adams
> On Jul 12, 2016, at 8:08 AM, Gianfranco Costamagna > wrote: > > sponsoring soon! > > thanks for your contribution to Debian! Thank you for sponsoring my package Gianfranco!

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-12 Thread Gianfranco Costamagna
Hi, > >Aren't there different rationales for source and binary package names? >This is what I have been thinking: > >- source package name should be upstream's name, because it's the > *source* >- binary package name is whatever makes most sense for someone typing > apt-get Indeed, I didn't

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-11 Thread Sean Whitton
On Mon, Jul 11, 2016 at 05:13:22PM +, Gianfranco Costamagna wrote: > src:setcolortemperature > binary: sct > > please choose one, and use the same, you can't usually expect users to install > a package and expect a binary called in another way. > > I seem to be pedantic, but I really like to

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-11 Thread Gianfranco Costamagna
Hi, >K == Kelvin, not kilo. this is what happens when one person is not using google :) thanks you all! G.

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-11 Thread Jakub Wilk
* Gianfranco Costamagna , 2016-07-11, 17:13: are you sure it is 6500K? 6500K should be equal to "650" K == Kelvin, not kilo. -- Jakub Wilk

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-11 Thread Andrew Shadura
On 11 Jul 2016 19:18, "Gianfranco Costamagna" wrote: > > Hi, > > >This has been fixed. Now when -h is passed usage is printed and if the > > >temperature passed is wrong usage will also be printed. > > > I still see nothing when called with no parameters :) > > (well you

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-11 Thread Gianfranco Costamagna
Hi, >This has been fixed. Now when -h is passed usage is printed and if the >temperature passed is wrong usage will also be printed. I still see nothing when called with no parameters :) (well you might print something like "temperature reset to the default value (65K) or similar) BTW If no

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-08 Thread Jacob Adams
control: retitle -1 RFS: setcolortemperature/1.3-1 ITP On 07/08/2016 12:29 PM, Gianfranco Costamagna wrote: > > the package is quite simple, but I would appreciate something more > verbose when calling it with wrong parameters. > e.g. > sct > sct -h > sct -v > sct 10 > sudo sct 10 > sudo sct

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-08 Thread Gianfranco Costamagna
control: owner -1 ! control: tags -1 moreinfo >Thanks for all your help reviewing my package! the package is quite simple, but I would appreciate something more verbose when calling it with wrong parameters. e.g. sct sct -h sct -v sct 10 sudo sct 10 sudo sct 14 all gives no output. After

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Jacob Adams
control: tag -1 -moreinfo On 07/07/2016 10:38 PM, Sean Whitton wrote: > You didn't bump it in the Debian changelog :) > > I consider this package ready to upload to Debian (packaging repo commit > 6f964da0, main repo commit 00b97fee), except for: > > - fix version in the Debian changelog > -

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Sean Whitton
control: tag -1 +confirmed control: noowner -1 Hello, On Thu, Jul 07, 2016 at 10:21:40PM -0400, Jacob Adams wrote: > Should be fixed now. > I just signed the 1.1 tarball. > I'm not sure why uscan was trying to fetch that anyway as I bumped the > version number to 1.2 after my last changes. You

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Jacob Adams
On 07/07/2016 10:13 PM, Sean Whitton wrote: > Hello, > > Unfortunately your watch file doesn't seem to work now. > > uscan warn: In directory ., downloading > > https://github.com/Tookmund/setcolortemperature/releases/download/v1.1/setcolortemperature-1.1.tar.gz.asc > failed: 404 Not Found

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Sean Whitton
Hello, Unfortunately your watch file doesn't seem to work now. uscan warn: In directory ., downloading https://github.com/Tookmund/setcolortemperature/releases/download/v1.1/setcolortemperature-1.1.tar.gz.asc failed: 404 Not Found -- Sean Whitton

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Jacob Adams
control: retitle -1 RFS: setcolortemperature/1.2-1 ITP On 07/07/2016 02:00 AM, Sean Whitton wrote: > Hello, > > Since you are upstream, would you consider providing a changelog that > you can install? Lintian is saying no-upstream-changelog and it seems > we can easily fix that :) > I added a

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-07 Thread Sean Whitton
Hello, Since you are upstream, would you consider providing a changelog that you can install? Lintian is saying no-upstream-changelog and it seems we can easily fix that :) -- Sean Whitton

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-06 Thread Jacob Adams
On 07/05/2016 08:02 AM, Sean Whitton wrote: > control: owner -1 ! > control: tag -1 +moreinfo > > Dear Jacob, > > This looks like a nice alternative to redshift-gtk. Thanks for > packaging it. I can't sponsor the upload, but I hope this review is > useful to you. Thanks! The review definitely

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-06 Thread Sean Whitton
Hello Peter and Jacob, On Tue, Jul 05, 2016 at 11:19:55PM +0300, Peter Pentchev wrote: > Actually I don't see a problem with the machine-readable copyright > specification here; if you're referring to the fact that the contents > of the "Copyright" field is not in the usual "list of " > format,

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-06 Thread Jacob Adams
On 07/05/2016 07:46 PM, Sean Whitton wrote: > Hello Peter and Jacob, > > On Tue, Jul 05, 2016 at 11:19:55PM +0300, Peter Pentchev wrote: >> Actually I don't see a problem with the machine-readable copyright >> specification here; if you're referring to the fact that the contents >> of the

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-06 Thread Peter Pentchev
On Tue, Jul 05, 2016 at 12:02:53PM +, Sean Whitton wrote: [snip] > 8. This doesn't make sense (doesn't follow DEP-5 machine-readable >copyright file format) -- please check: > > Files: sct.c > Copyright: 2016 Ted Unangst >whitepoints data copyright

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-07-05 Thread Sean Whitton
control: owner -1 ! control: tag -1 +moreinfo Dear Jacob, This looks like a nice alternative to redshift-gtk. Thanks for packaging it. I can't sponsor the upload, but I hope this review is useful to you. 1. Could you explain why you are packaging your fork rather then the original? (This

Bug#829151: RFS: setcolortemperature/1.1-1 ITP

2016-06-30 Thread Jacob Adams
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package "setcolortemperature" * Package name: setcolortemperature Version : 1.1-1 Upstream Author : Ted Unangst (I am upstream maintainer though) * URL