Create a gist now

Instantly share code, notes, and snippets.

Make Jekyll play nice with Windows again post Albino + posix-spawn refactor
diff --git a/lib/albino.rb b/lib/albino.rb
index 387c8e9..b77d55e 100644
--- a/lib/albino.rb
+++ b/lib/albino.rb
@@ -1,4 +1,5 @@
require 'posix-spawn'
+require 'rbconfig'
##
# Wrapper for the Pygments command line tool, pygmentize.
@@ -84,11 +85,21 @@ class Albino
proc_options[:timeout] = options.delete(:timeout) || self.class.timeout_threshold
command = convert_options(options)
command.unshift(bin)
- Child.new(*(command + [proc_options.merge(:input => write_target)]))
+ if RbConfig::CONFIG['host_os'] =~ /(mingw|mswin)/
+ output = ''
+ IO.popen(command, mode='r+') do |p|
+ p.write @target
+ p.close_write
+ output = p.read.strip
+ end
+ output
+ else
+ Child.new(*(command + [proc_options.merge(:input => write_target)]))
+ end
end
def colorize(options = {})
- out = execute(options).out
+ out = RbConfig::CONFIG['host_os'] =~ /(mingw|mswin)/ ? execute(options) : execute(options).out
# markdown requires block elements on their own line
out.sub!(%r{</pre></div>\Z}, "</pre>\n</div>")
@hickford
hickford commented May 9, 2012

Stupid question, how do I apply the patch?

@madhur
Owner
madhur commented May 10, 2012
@hickford

I reported the bug that this patch fixes in Albino github-archive/albino#14

Hopefully the patch will be accepted

@codeimpossible

@madhur thanks for posting this up here, patch worked perfectly. Shame that albino seems to be abandoned though.

@yanhaijing

thanks,but if i have not albino-1.3.3,。。。

@yanhaijing

thanks,but if i have not albino-1.3.3,。。。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment