Re: RFS: ruby-fast-blank

2016-08-17 Thread Antonio Terceiro 
On Wed, Aug 17, 2016 at 07:45:38PM +0300, Hleb Valoshka wrote:
> On 8/17/16, Sruthi Chandran  wrote:
> 
> > Hi,I prepared the packaging of ruby-fast-blank. It is lintian cleanand
> > I tested it with build-and-upload script from pkg-ruby-extras repo.
> >
> > As my membership request to alioth is still pending, I've added my
> > packageto the following
> > git-repo:https://git.fosscommunity.in/debian-ruby/ruby-fast-blank
> >
> > Please review and upload.
> 
> I appreciate you intent to contribute to Debian, but do we really need
> this packaged? For me this gem is too trivial, and other gems which
> may use it should be patched not to use it.
> 
> class String; def blank?; self.strip.empty?; end; end
> 
> Does it worth to have its own gem?

I do agree we have to push back against frivolous dependencies, and
bindling packaging everything just because it is listed as dependency
for something that you actually want.

fast_blank however seems to be something created with a purpose,
though ...


signature.asc
Description: PGP signature

Re: RFS: ruby-fast-blank

2016-08-17 Thread Hleb Valoshka 
On 8/17/16, Sruthi  wrote:
> I packaged the gem because it is listed as one of the dependencies for
> Discourse.

Why just not to patch Discourse?

Re: RFS: ruby-fast-blank

2016-08-17 Thread Hleb Valoshka 
On 8/17/16, Sruthi Chandran  wrote:

> Hi,I prepared the packaging of ruby-fast-blank. It is lintian cleanand
> I tested it with build-and-upload script from pkg-ruby-extras repo.
>
> As my membership request to alioth is still pending, I've added my
> packageto the following
> git-repo:https://git.fosscommunity.in/debian-ruby/ruby-fast-blank
>
> Please review and upload.

I appreciate you intent to contribute to Debian, but do we really need
this packaged? For me this gem is too trivial, and other gems which
may use it should be patched not to use it.

class String; def blank?; self.strip.empty?; end; end

Does it worth to have its own gem?

I did not patched ruby-buff-* usage in my packages only because
ruby-buff-* was already packaged.