[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Outreachy project



On Wed, Mar 27, 2019 at 04:30:13PM -0700, jyzhou15 wrote:
> "PS:"
> I've committed an autopkgtest for clearcut
> (https://salsa.debian.org/med-team/clearcut/blob/master/debian/tests/run-unit-test)
> to master.

Good.  Thanks a lot!  I see you understood the principle!
(I've just added an interpreter line to the shell script.)

Please also make sure you add an entry in debian/changelog closing the
bug.  If there are other changes that are not yet uploaded feel free to
"take over" the changelog paragraph with your ID.  I'd love to give my
outreachy students to "own" that upload.

BTW, if you see some chance to verify against "correct" results as
discussed in the other mail feel free to add a `diff ...` .

> I also noticed that the readme for clearcut is inaccurate (mentions a
> "make install" but that target doesn't exist).

Well spotted.  However, that part of the upstream README about
installation is not relevant for Debian users anyway since if a user
might read this on /usr/share/doc/clearcut/README.gz the software is
actually installed.  Users are trained to understand to ignore
installation instructions about software that was installed by the
Debian package.

However, you might report this issue upstream (may be if you contact
about valid results) since it is relevant for users who are not
installing the software via Debian package.

Just tell me once you consider the test finished (I'd consider bug
#909711 fixed by your test - but may be you want to gain for more
and I want *you* to be happy with it) and I'll upload the package.

Thanks a lot for your contribution

     Andreas.

-- 
http://fam-tille.de


Reply to: