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

Bug#998178: ITP: martchus-cpp-utilities -- C++ utility classes for Syncthing Tray



Hi Hannah,

Please prioritise this package, since it's a build dependency of
martchus-qtutilities.  I'm sorry for having treated them as coequal when
they're not--this was my faulty assumption/oversight.

Hannah Rittich <void@rittich.net> writes:

> Hi Nicholas,
>
> On 21/11/2021 22:12, Nicholas D Steeves wrote
>> I've fixed all of these issues on my local branch, so if you find
>> this stuff boring I can take care of it.  If you're considering
>> pursuing Debian Maintainer (uploading) status it's important to learn
>> why these are significant.
>
> I do not mind doing these changes. If you, however, have already made
> them, it is rather pointless for me to do them again. But thanks for the
> heads-up.
>

Well, I didn't commit anything, having 'just made corrections when I
could find the time to, and then consulted the diff when I finally found
time to write a review email.  It's totally up to you whether or not you
want to participate in the didactic review/revise process.

>> Is there any reason you've chosen debhelper-compat 12 rather than
>> 13? If not, please use 13.
>
> No. I just copied that from some tutorial. ;)
>

I totally understand and did this too back in the day ;)

> On 22/11/2021 00:57, Nicholas D Steeves wrote:
>>> * source:Description does not yet appear to be supported by Policy
>>> (see bug #998165).  I think you're prioritising uploading this
>>> package rather than evolving Debian Policy, so I'd recommend
>> 
>> ... using the old copy & paste method for now, and reintroducing 
>> substvars once that Policy bug is resolved.
>
> You could just revert my last commit. Using the X-Description variable
> is accepted by lintian.
>

This isn't the nature of the issue.  Please see today's reply to the
martchus-qtutilities bug.

Best,
Nicholas

Attachment: signature.asc
Description: PGP signature


Reply to: