Re: RFS: qtltools -- Tool set for molecular QTL discovery and analysis
Hi Afif, 2016-12-02 10:33 GMT+01:00 Dylan : > Hi Afif, > > Thanks for your review and your useful comments :-) > > 2016-12-02 10:11 GMT+01:00 Afif Elghraoui : >> >> Many thanks for preparing this package. My comments: >> >> * Rather than patch the include statements for libeigen in the source, >> it is more sustainable to append -I/usr/include/eigen3 to CPPFLAGS in >> debian/rules (using DEB_CPPFLAGS_MAINT_APPEND). This will save you >> maintenance of a patch and provide more flexibility. > >> * very minor: the folder debian/upstream.docs has data inside it rather >> than documentation. Would you clarify this in README.source or use a >> more intuitive name? >> I have updated the package according to your comments. Thanks Best regards, Dylan
Re: RFS: qtltools -- Tool set for molecular QTL discovery and analysis
على الجمعـة 2 كانون الأول 2016 01:33، كتب Dylan: >> * I've been told by ftpmasters before that, if the authors simply say >> > "GPL" or provide no explicit GPL license statement besides simply >> > bundling the text of GPL-3, the license is to be recorded as /any/ GPL >> > version, i.e., GPL-1+. If we upload in the current state, we might get a >> > rejection for this reason. >> > > Maybe, I miss something but the authors specify in the header of each > file that the license is GPL-3+: > "...either version 3 of the License, or (at your option) any later > version...". > > Ah, you're right. I thought I did this particular check yesterday and so made the comment from memrory, but I must have been looking at another package or something. Thanks and regards Afif -- Afif Elghraoui | عفيف الغراوي http://afif.ghraoui.name
Re: RFS: qtltools -- Tool set for molecular QTL discovery and analysis
Hi Afif, Thanks for your review and your useful comments :-) 2016-12-02 10:11 GMT+01:00 Afif Elghraoui : > > Many thanks for preparing this package. My comments: > > * Rather than patch the include statements for libeigen in the source, > it is more sustainable to append -I/usr/include/eigen3 to CPPFLAGS in > debian/rules (using DEB_CPPFLAGS_MAINT_APPEND). This will save you > maintenance of a patch and provide more flexibility. > I will update this point. > * I've been told by ftpmasters before that, if the authors simply say > "GPL" or provide no explicit GPL license statement besides simply > bundling the text of GPL-3, the license is to be recorded as /any/ GPL > version, i.e., GPL-1+. If we upload in the current state, we might get a > rejection for this reason. > Maybe, I miss something but the authors specify in the header of each file that the license is GPL-3+: "...either version 3 of the License, or (at your option) any later version...". > * very minor: the folder debian/upstream.docs has data inside it rather > than documentation. Would you clarify this in README.source or use a > more intuitive name? > I will update this point as well. > the build is taking a bit too long on the computer I'm using now, so I > would have to try on a more resourceful machine to fully test it out. > Thanks :-) Best regards, Dylan
Re: RFS: qtltools -- Tool set for molecular QTL discovery and analysis
Hi, Dylan, على الثلاثاء 29 تشرين الثاني 2016 14:01، كتب Dylan: > Hi, > > Package name: qtltools > URL: https://qtltools.github.io/qtltools/ > License: GPL-3+ > Description: Tool set for molecular QTL discovery and analysis > QTLtools is a tool set for molecular Quantitative Trait Loci (QTL) discovery > and analysis. It allows user to go from the raw sequence data to collection > of > molecular QTL in few easy-to-perform steps. QTLtools contains multiple > methods > to prepare the data, to discover proximal and distal molecular QTL and to > finally integrate them with GWAS variants and functional annotations of the > genome. > > > The package is ready for the review and the upload. > Many thanks for preparing this package. My comments: * Rather than patch the include statements for libeigen in the source, it is more sustainable to append -I/usr/include/eigen3 to CPPFLAGS in debian/rules (using DEB_CPPFLAGS_MAINT_APPEND). This will save you maintenance of a patch and provide more flexibility. * I've been told by ftpmasters before that, if the authors simply say "GPL" or provide no explicit GPL license statement besides simply bundling the text of GPL-3, the license is to be recorded as /any/ GPL version, i.e., GPL-1+. If we upload in the current state, we might get a rejection for this reason. * very minor: the folder debian/upstream.docs has data inside it rather than documentation. Would you clarify this in README.source or use a more intuitive name? the build is taking a bit too long on the computer I'm using now, so I would have to try on a more resourceful machine to fully test it out. Thanks and regards Afif -- Afif Elghraoui | عفيف الغراوي http://afif.ghraoui.name