Re: [MoM] bandage
Hi Cédric,
thanks for pushing.
On Tue, Aug 15, 2017 at 11:25:33AM +0000, Cédric Lood wrote:
> > MoM is about directing you quickly to the fine manuals and nobody will
> > blame you about trivial questions which frequently are not that tivial.
> > I'd like to smoothen your entry and shorten the time you need to dig
> > for proper information.
>
> You are right of course, the debian project has an insane amount of documentation and I keep being sidetracked when reading it :)
Yes, MoM should be your personal shortcut through all this. ;-)
> > When I checked it out I have not yet seen any debian/ directory.
>
> I have pushed a few commits just now, if you wanna have a look and will file a new ITP later today. As guideline, I am following very closely the Debian Med policy, as well as the guide for new maintainers.
I have checked your packaging and have some comments:
debian/upstream/metadata
I simply pushed a syntax error and extended the authors
field which is the one that is actively used on our tasks
pages[1]
debian/changelog
when using the suggested Debian Med template you need to
replace some strings like the date stamp manually by
the result of `date -R`. You also need to add the package
name and the version. Otherwise any attempt to build the
package will fail due to invalid d/changelog.
debian/copyright
I simplified this file and pushed. The general logic is
that you start with some kind of general copyright paragraph
featuring all files
Files: *
and afterwards you specify exceptions from this.
The files is just missing two paragraphs for the used
licenses which should specify the location of the full
copies of the according licenses. (Pick your favourite
package as example how to specify this or ask back here
if it might be not clear what I mean.)
debian/doc-base:
This template can be most probably be removed. I do not
see that the software contains any extensive documentation.
debian/get-orig-source:
Please remove this template since debian/watch should work.
The script is only used if upstream does not follow sensible
rules to provide a versioned tarball.
debian/watch
Please remove all commented lines except the one for Github
releases and make this point to
https://github.com/rrwick/Bandage/releases
Just a hint: The debian/watch file is the first thing I'm
fixing to get the upstream source right via uscan to see
whether it can correctly download the source.
debian/rules
After you have fixed everything (specifically the changelog
file) you can try to build the package. You should read the
file README.md from upstream how to build. It requires qmake
which you need to specify in debian/control as a Build-Depends.
Hope these hints are helpful and enable you to continue easily. Feel
free to ask here about any detail that might remain unclear.
Kind regards
Andreas.
[1] https://blends.debian.org/med/tasks/bio
--
http://fam-tille.de
Reply to: