Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-24 Thread Ghislain Vaillant
oid> *From*:"Ghislain Vaillant" <ghisv...@gmail.com> *Date*:Tue, 22 Sep, 2015 at 19:33 *Subject*:Bug#799768: RFS: ismrmrd/1.3.1-1 On 22/09/15 17:32, Gianfranco Costamagna wrote: > I would appreciate a lintian wa

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Gianfranco Costamagna
I can sponsor the package with that lintian warning. as said before the package should be lintian clean as long as lintian is correct. But overriding lintian is wrong, because it is still an issue. If it is ok for you I would prefer a package with a lintian warning. cheers, Gianfranco

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Gianfranco Costamagna
> >I still don't get what the recommended solution is. So far, I have been >(wrongly?) following what Lintian suggested to do with embedded jquery. I would appreciate a lintian warning instead of a broken documentation e.g. look at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=736360

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Ghislain Vaillant
On 22/09/15 17:32, Gianfranco Costamagna wrote: I would appreciate a lintian warning instead of a broken documentation e.g. look at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=736360 G. Should the warning be overridden for the purpose or the review? Apart from that, I am ready to

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Ghislain Vaillant
On 22/09/15 14:57, Gianfranco Costamagna wrote: Control: merge -1 799767 Control: owner -1 ! Hi Ghislain, the packaging looks good, however I have some nitpicks/issues: rules file: "-DCMAKE_INSTALL_PREFIX=/usr" is this really needed? AFAIK it should be the default inside a dh build.

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Ghislain Vaillant
Package: sponsorship-requests Severity: normal Dear mentors, I am looking for a sponsor for my package "ismrmrd" * Package name: ismrmrd Version : 1.3.1-1 Upstream Author : The ISMRMRD developers * URL : http://ismrmrd.github.io/ * License : Expat Section

Bug#799768: RFS: ismrmrd/1.3.1-1

2015-09-22 Thread Gianfranco Costamagna
Control: merge -1 799767 Control: owner -1 ! Hi Ghislain, the packaging looks good, however I have some nitpicks/issues: rules file: "-DCMAKE_INSTALL_PREFIX=/usr" is this really needed? AFAIK it should be the default inside a dh build. (please check) "--builddirectory=$(BUILDDIR)" same