Bug#891892: lintian: False positive: statically-linked-binary despite the Build-Depends on golang-go

2018-03-13 Thread Chris Lamb
Hi Arnaud,

> > $built_with_golang
> >   = $src->info->relation('build-depends')
> >   ->implies('golang-go | golang-any');
> >
> > Not sure whether it's the >= version of the "golang-go" part or
> > the "golang-1.9-go" part in itself is causing it not to match..
> 
> These lines do not raise a warning
> 
>   golang-any (>= 2:1.9~)
>   golang-go (>= 2:1.9~)
> 
> However this one raise a warning
> 
>   golang-any | golang-1.9-go

Ah, sorry, I should/could have told you that :(  Sorry for causing you
extra work..

> let me just clarify that with the go packaging team, and fix the control
> file to start with.

Can you clarify? As in, shall we close this bug?


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#891892: lintian: False positive: statically-linked-binary despite the Build-Depends on golang-go

2018-03-13 Thread Arnaud
Hi,

I did a few tests !


On 03/02/2018 09:20 PM, Chris Lamb wrote:
> Hi Arnaud,
>
>> The statically-linked-binary warninf shouldn't be there, as the field
>> Build-Depends contains:
>>
>> golang-go (>= 2:1.9~) | golang-1.9-go
> The Lintian code in question is:
>
> $built_with_golang
>   = $src->info->relation('build-depends')
>   ->implies('golang-go | golang-any');
>
> Not sure whether it's the >= version of the "golang-go" part or
> the "golang-1.9-go" part in itself is causing it not to match..

These lines do not raise a warning

  golang-any (>= 2:1.9~)
  golang-go (>= 2:1.9~)

However this one raise a warning

  golang-any | golang-1.9-go

And this last one, more interesting, fails the build:

  golang-1.9-go

Actually, only 'golang-go' installs the binary '/usr/bin/go' (which is
just a symlink). Installing only 'golang-1.x-go'  is not enough, so let
me just clarify that with the go packaging team, and fix the control
file to start with.

Cheers

  Arnaud



Bug#886096: lintian: Emit warnings for Alioth URLs in packaging (migration to Salsa)

2018-03-13 Thread Chris Lamb
Hi Stuart,

> we now only have 6 time quanta until alioth is read-only

I like this "time quanta" concept :)  Okay, agreed, let's get onto
this ASAP.

Can you help by drafting the specific tag (or tags?) and giving some
examples of Vcs-Foo it should trigger on?


Best wishes,

-- 
Chris Lamb
chris-lamb.co.uk / @lolamby



Bug#886096: lintian: Emit warnings for Alioth URLs in packaging (migration to Salsa)

2018-03-13 Thread Stuart Prescott
Control: tags 886096 - moreinfo

> > lintian: Emit warnings for Alioth URLs in packaging
> 
> I'm going to "park" this bug for the time being until we know the state
> of any redirection services — simply warning maintainers right now that
> they are using anonscm URIs is probably not tremendously helpful.

I think all the concerns and questions raised in this bug are now addressed.


salsa.debian.org is now out of beta and the threat of database drops has gone 
[1]; it is widely adopted by the project and impending close-down date of 
alioth draws closer [2].

It is now documented that the redirector is only for http/https URLs and it is 
not intended to be a replacement for changing the Vcs-* fields within debian/
control [3]. That ssh and git protocol connections are not redirected means 
that anonscm URLs from debian/control will play badly with existing magic from 
debcheckout or local pushInsteadOf config that was used to make those URLs 
usable for developers.

Maintainers need to move their repositories either individually or as part of 
teams and the lintian message could point towards a suitable tool like Mehdi's 
salsa-scripts to do this [4].

I think we're at the point where lintian should be giving very strong guidance 
on this matter. It's quite obviously not an ftp-master reject, but a very in-
the-face warning or error is appropriate. In a project where time is normally 
quantised in 'week' units and several quanta are normally allowed for any 
action, we now only have 6 time quanta until alioth is read-only.


[1] https://lists.debian.org/debian-devel-announce/2018/01/msg4.html

[2] Wheezy EOL, 2018-04-25 https://wiki.debian.org/
Alioth#Deprecation_of_Alioth https://wiki.debian.org/DebianWheezy

[3] https://salsa.debian.org/salsa/AliothRewriter/blob/master/README.md

[4] https://salsa.debian.org/mehdi/salsa-scripts/

-- 
Stuart Prescotthttp://www.nanonanonano.net/   stu...@nanonanonano.net
Debian Developer   http://www.debian.org/ stu...@debian.org
GPG fingerprint90E2 D2C1 AD14 6A1B 7EBB 891D BBC1 7EBB 1396 F2F7



Processed: Re: lintian: Emit warnings for Alioth URLs in packaging (migration to Salsa)

2018-03-13 Thread Debian Bug Tracking System
Processing control commands:

> tags 886096 - moreinfo
Bug #886096 [lintian] lintian: Emit warnings for Alioth URLs in packaging 
(migration to Salsa)
Removed tag(s) moreinfo.

-- 
886096: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=886096
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems