Re: [pkg-go] Comments regarding zap_1.5.0+git20170802.3.e68420e-1_amd64.changes

2017-09-06 Thread Pirate Praveen
On ബുധന്‍ 06 സെപ്റ്റംബര്‍ 2017 05:15 വൈകു, Shengjing Zhu wrote:
> (not from ftp-master, just team member of pkg-go, and not cc ftp-master)
> 
> Why ship this /usr/bin/zap-readme? I don't think this is a useful
> binary for users.

ok, We can remove it from next upload.

> the readme binary is generate from
> https://github.com/uber-go/zap/blob/master/internal/readme/readme.go
> which is in an internal directory.
> And after looked at the code roughly, I think this is used by library
> mantainer, not users, which is to generate the
> https://github.com/uber-go/zap/blob/master/README.md

my main motivation was to get the library part for gitaly, so did not
dig deeper about the binary. Thanks for looking deeper.



signature.asc
Description: OpenPGP digital signature
___
Pkg-go-maintainers mailing list
Pkg-go-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers

Re: [pkg-go] Comments regarding zap_1.5.0+git20170802.3.e68420e-1_amd64.changes

2017-09-06 Thread Shengjing Zhu
(not from ftp-master, just team member of pkg-go, and not cc ftp-master)

Why ship this /usr/bin/zap-readme? I don't think this is a useful
binary for users.

the readme binary is generate from
https://github.com/uber-go/zap/blob/master/internal/readme/readme.go
which is in an internal directory.
And after looked at the code roughly, I think this is used by library
mantainer, not users, which is to generate the
https://github.com/uber-go/zap/blob/master/README.md


On Wed, Sep 6, 2017 at 7:00 PM, Pirate Praveen  wrote:
> On ശനി 02 സെപ്റ്റംബര്‍ 2017 12:57 വൈകു, Luca Falavigna wrote:
>> Hi,
>>
>> /usr/bin/readme is too generic to be placed under a location defined in PATH.
>> Would it be possible to use a more specialized name, such as zap-readme?
>>
>
> done.
>> Cheers,
>> Luca
>>
>>
>>
>> ___
>> Pkg-go-maintainers mailing list
>> Pkg-go-maintainers@lists.alioth.debian.org
>> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers
>>
>
>
>
> ___
> Pkg-go-maintainers mailing list
> Pkg-go-maintainers@lists.alioth.debian.org
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers



-- 
Best regards,
Shengjing Zhu

___
Pkg-go-maintainers mailing list
Pkg-go-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers

Re: [pkg-go] Comments regarding zap_1.5.0+git20170802.3.e68420e-1_amd64.changes

2017-09-06 Thread Pirate Praveen
On ശനി 02 സെപ്റ്റംബര്‍ 2017 12:57 വൈകു, Luca Falavigna wrote:
> Hi,
> 
> /usr/bin/readme is too generic to be placed under a location defined in PATH.
> Would it be possible to use a more specialized name, such as zap-readme?
>

done.
> Cheers,
> Luca
> 
> 
> 
> ___
> Pkg-go-maintainers mailing list
> Pkg-go-maintainers@lists.alioth.debian.org
> http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers
> 




signature.asc
Description: OpenPGP digital signature
___
Pkg-go-maintainers mailing list
Pkg-go-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-go-maintainers