Re: Bug#886399: RFS: opencascade/7.2.0-2 [ITP]
On Fri, 2018-01-05 at 13:30 +0100, Tobias Frost wrote:
> Hi Kurt Kremitzki,
> many thanks for the package!
> As the package is huge, I think review will take some time and
> some iterations. I will start on the review, but I hope that others
> will chime in. (I will not have enough time for a complete review
Indeed this is a huge package, so thanks for taking the time to review
> - d/README.Debian.html
> I'm not sure if we want a html file as README.debian. It seems also
> quite outdates (referring to version 6.5). Its content is also
> by other parts of the pacakging (e.g by d/control), so I guess it
> should be deleted
> - d/README.source
> Same here, quilt is standard nowardays, so it can be retired.
> I'm not sure about the repacking part, but if that is still true,
> the package version would needs to be a -dfsg anyway.
Yes, these were artifacts from version 6.5.0 and were meant to be
deleted. I've fixed that.
> - d/changelog
> Please keep at Debian revision -1 until this is sponsored.
> But as this is a reintroduction of an old package, you will
> still need to record all the changes you have made to the old
> (There is also extra blank lines below your entries)
Thanks, I wasn't sure how to upload changes so I was bumping the Debian
revision number. I got help in #debian-mentors and corrected this.
> - d/compat: Did you see if you can use compat level 11?
Corrected as well.
> - please add dep3 headers to the patches.
I've added these, hopefully they look OK.
> - d/control
> - VCS-* are for the Debian packaging, not for the upstream
> - Would be great if you could maintain the package on a git repo;
> can use salsa.debian.org for it.
Ah, I wasn't sure on this point. I've uploaded the source to https://s
alsa.debian.org/kkremitzki-guest/opencascade and updated the VCS-*
stuff to point there. Does that look alright? I got started reading
the documentation for git-buildpackage but hopefully a full transition
to getting this package based on salsa.d.o instead of an upstream
tarball can be done as an improvement after getting the package into
> I try to dedicate some more time later...
Thanks again! The updated package is uploaded now at https://mentors.d