[DRE-maint] Bug#942596: Patch available by Fedora

[Antonio Terceiro]

On Mon, Apr 27, 2020 at 02:07:58PM +0200, Daniel Leidert wrote:
> tags 942596 + patch
> thanks
> 
> Hi there,
> 
> it seems Fedora people applied a patch to remove bundler usage from jekyll and
> make --skip-bundle behavior the default. Shall we use it too?
> 
> https://src.fedoraproject.org/rpms/rubygem-jekyll/blob/f30/f/0000-jekyll-commands-remove-bundle-install-step-for-new-c.patch

Yes. Although I would not remove the --skip-bundle option as that can
break scripts out there, and instead just assign `options["skip-bundle"]
= true` before doing anything (untested):

----------------8<----------------8<----------------8<-----------------
diff --git a/lib/jekyll/commands/new.rb b/lib/jekyll/commands/new.rb
index 09cee79..a85dbcb 100644
--- a/lib/jekyll/commands/new.rb
+++ b/lib/jekyll/commands/new.rb
@@ -16,6 +16,7 @@ module Jekyll
             c.option "skip-bundle", "--skip-bundle", "Skip 'bundle install'"
 
             c.action do |args, options|
+              options["skip-bundle"] = true
               Jekyll::Commands::New.process(args, options)
             end
           end
----------------8<----------------8<----------------8<-----------------

you would also need the changes to the test suite I think.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/pkg-ruby-extras-maintainers/attachments/20200427/8d4cd073/attachment-0001.sig>