Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-05-01 Thread Christoph Biedl
Shengjing Zhu wrote...

> I planned to merge this when shfmt has a new upstream release.
> However it seems more urgent now since it was removed from testing the
> last few days.

Thanks for the swift response. I'm glad to see shfmt will return to a
good shape soon.

Christoph.





signature.asc
Description: PGP signature


Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-04-29 Thread Nilesh Patra



Hi Christoph,

On 29 April 2022 5:47:01 pm IST, Christoph Biedl 
 wrote:
>Marcos Talau wrote...
>
>> I will make the package orphaned. Feel free to manage it.
>
>... but then nothing happened. Today I found shfmt and consider it a
>useful tool, hence I'm interested to see it in Debian, and in good
>state - which shfmt is no longer as it dropped out of testing.
>
>So, are you still interested in having it under the Go packaging team
>umbrella?

Speaking for myself: with the sort of hostile  response I've got from the 
former maintainer despite being nice to them and the lack of communication at 
their end, I've no interest in working on this further, and wouldn't reply to 
any mail on the same.

>As a last resort, but no earlier, I might take maintainership
>as well. But it will be outside the team then.

Sure, please go ahead with the same if you feel like.
One request though: please also create a (library)-dev binary package for the 
same. I'll then ask for a removal for golang-mvdan-sh which would effectively 
help close this bug which we are talking at.

Best, Nilesh



Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-04-29 Thread Shengjing Zhu
Hi,

On Fri, Apr 29, 2022 at 8:25 PM Christoph Biedl
 wrote:
>
> Marcos Talau wrote...
>
> > I will make the package orphaned. Feel free to manage it.
>
> ... but then nothing happened. Today I found shfmt and consider it a
> useful tool, hence I'm interested to see it in Debian, and in good
> state - which shfmt is no longer as it dropped out of testing.
>
> So, are you still interested in having it under the Go packaging team
> umbrella? As a last resort, but no earlier, I might take maintainership
> as well. But it will be outside the team then.
>

I planned to merge this when shfmt has a new upstream release.
However it seems more urgent now since it was removed from testing the
last few days.

-- 
Shengjing Zhu



Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-04-29 Thread Christoph Biedl
Marcos Talau wrote...

> I will make the package orphaned. Feel free to manage it.

... but then nothing happened. Today I found shfmt and consider it a
useful tool, hence I'm interested to see it in Debian, and in good
state - which shfmt is no longer as it dropped out of testing.

So, are you still interested in having it under the Go packaging team
umbrella? As a last resort, but no earlier, I might take maintainership
as well. But it will be outside the team then.

Christoph


signature.asc
Description: PGP signature


Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-23 Thread Marcos Talau
Hi there,

I don't intend to integrate the team now. Considering the strong and continuous
harassment to join the team or to add co-maintainers since my initial work,
I will make the package orphaned. Feel free to manage it.

Have a nice day.


Cheers,
mt


signature.asc
Description: PGP signature


Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-22 Thread Nilesh Patra
On Wed, Mar 23, 2022 at 04:04:41AM +0800, Shengjing Zhu wrote:
> It's upstream that puts them in one repo.
> And src:shfmt clearly contains all the source of src:golang-mvdan-sh
> 
> > Also, if you look even "gosh" is a part of mvdan-sh and it is also in the 
> > archive as a separate package;
> > so if you are merging, even gosh should be merged?
> 
> You mean bin:gauche package which provides /usr/bin/gosh? It's
> obviously a different source.

ACK, I overlooked that. Looks like shfmt is the useful tool here, so if Marcos 
could
transfer his repo into the
golang team, we can proceed with the merge.

Regards,
Nilesh


signature.asc
Description: PGP signature


Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-22 Thread Shengjing Zhu
On Wed, Mar 23, 2022 at 4:04 AM Shengjing Zhu  wrote:
>
> On Wed, Mar 23, 2022 at 3:57 AM Nilesh Patra  wrote:
> >
> > Hi Shengjing,
> >
> > On Wed, 23 Mar 2022 02:35:20 +0800 Shengjing Zhu  wrote:
> > > Control: found -1 shfmt/3.4.3-1
> > > Control: found -1 golang-mvdan-sh/3.4.3+ds-1
> >^^
> >
> > I have excluded the source for shfmt in that version[1]
> > So I do not think duplicated source is true anymore?
> >
> > [1]: 
> > https://salsa.debian.org/go-team/packages/golang-mvdan-sh/-/blob/debian/sid/debian/copyright#L5
> >
> > > The two packages are same source, could you merge them?
> >
> > I am not fully certain if it is a good idea, since shfmt is already 
> > available as a git repo,
> > and also as a Debian package, so I do not follow as to why this should be a 
> > monorepo?
> >
>
> It's upstream that puts them in one repo.
> And src:shfmt clearly contains all the source of src:golang-mvdan-sh
>
> > Also, if you look even "gosh" is a part of mvdan-sh and it is also in the 
> > archive as a separate package;
> > so if you are merging, even gosh should be merged?
>
> You mean bin:gauche package which provides /usr/bin/gosh? It's
> obviously a different source.
>

Just be much clearer, merging means merging the two source packages,
the binary packages shouldn't be merged.

-- 
Shengjing Zhu



Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-22 Thread Shengjing Zhu
On Wed, Mar 23, 2022 at 3:57 AM Nilesh Patra  wrote:
>
> Hi Shengjing,
>
> On Wed, 23 Mar 2022 02:35:20 +0800 Shengjing Zhu  wrote:
> > Control: found -1 shfmt/3.4.3-1
> > Control: found -1 golang-mvdan-sh/3.4.3+ds-1
>^^
>
> I have excluded the source for shfmt in that version[1]
> So I do not think duplicated source is true anymore?
>
> [1]: 
> https://salsa.debian.org/go-team/packages/golang-mvdan-sh/-/blob/debian/sid/debian/copyright#L5
>
> > The two packages are same source, could you merge them?
>
> I am not fully certain if it is a good idea, since shfmt is already available 
> as a git repo,
> and also as a Debian package, so I do not follow as to why this should be a 
> monorepo?
>

It's upstream that puts them in one repo.
And src:shfmt clearly contains all the source of src:golang-mvdan-sh

> Also, if you look even "gosh" is a part of mvdan-sh and it is also in the 
> archive as a separate package;
> so if you are merging, even gosh should be merged?

You mean bin:gauche package which provides /usr/bin/gosh? It's
obviously a different source.

-- 
Shengjing Zhu



Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-22 Thread Nilesh Patra
Hi Shengjing,

On Wed, 23 Mar 2022 02:35:20 +0800 Shengjing Zhu  wrote:
> Control: found -1 shfmt/3.4.3-1
> Control: found -1 golang-mvdan-sh/3.4.3+ds-1
   ^^

I have excluded the source for shfmt in that version[1]
So I do not think duplicated source is true anymore?

[1]: 
https://salsa.debian.org/go-team/packages/golang-mvdan-sh/-/blob/debian/sid/debian/copyright#L5

> The two packages are same source, could you merge them?

I am not fully certain if it is a good idea, since shfmt is already available 
as a git repo,
and also as a Debian package, so I do not follow as to why this should be a 
monorepo?

Also, if you look even "gosh" is a part of mvdan-sh and it is also in the 
archive as a separate package;
so if you are merging, even gosh should be merged?
Please let me know the reasons a bit more clearly.

> Marcos Talau, could you mind bring shfmt to Go package team umbrella?

(CC'ed Marcos explicitly again)
@Marcos, kindly do this

Regards,
Nilesh


signature.asc
Description: PGP signature


Bug#1008112: Source is duplicated with golang-mvdan-sh

2022-03-22 Thread Shengjing Zhu
Source: shfmt, golang-mvdan-sh
Severity: normal
X-Debbugs-Cc: z...@debian.org

Control: found -1 shfmt/3.4.3-1
Control: found -1 golang-mvdan-sh/3.4.3+ds-1

Hi,

The two packages are same source, could you merge them?

Marcos Talau, could you mind bring shfmt to Go package team umbrella?