Bug#814829: [RFR] templates://eviacam/{templates}

2016-03-01 Thread Cesar Mauri
Hi, Dropping "software" works, but I'm not so sure about "camera", since by default that means the non-video ("still camera") variety. As long as nobody expects PCs to have integrated SLR cameras it should be fairly obvious what you mean, but it still might be safer to say: It works on a

Bug#814829: [RFR] templates://eviacam/{templates}

2016-02-29 Thread Cesar Mauri
Hi, Thanks you both Christian and Justin for your review. Your improvements look great to me! I've just changed webcam for camera and removed the trailing "software" after Facial Mouse. I created the path using "git diff". I hope that is fine. Cheers, C. Template:

Bug#814829: [RFR] templates://eviacam/{templates}

2016-02-27 Thread Justin B Rye
Cesar Mauri wrote: > Thanks you both Christian and Justin for your review. Your improvements look > great to me! > > I've just changed webcam for camera and removed the trailing "software" after > Facial Mouse. Dropping "software" works, but I'm not so sure about "camera", since by default

Bug#814829: [RFR] templates://eviacam/{templates}

2016-02-22 Thread Justin B Rye
Christian PERRIER wrote: > Rationale: > --- eviacam.old/debian/templates 2016-02-15 19:53:17.828347882 +0100 > +++ eviacam/debian/templates 2016-02-22 07:12:12.633368880 +0100 > @@ -2,11 +2,11 @@ > Type: boolean > Default: false > _Description: Should eviacamloader be installed 'setuid

Bug#814829: [RFR] templates://eviacam/{templates}

2016-02-21 Thread Christian PERRIER
Please find, for review, the debconf templates and packages descriptions for the eviacam source package. This review will last from Monday, February 22, 2016 to Thursday, March 03, 2016. Please send reviews as unified diffs (diff -u) against the original files. Comments about your proposed