Use sRGB patch by default.

This also applies some kind of convention for the available options.
`--srgb` is still defined for backwards compatibility, but it technically
does nothing.
This commit is contained in:
2012-03-19 23:02:25 +00:00
parent 8b7c2f5170
commit e507b7b031
2 changed files with 10 additions and 8 deletions

View File

@@ -30,11 +30,11 @@ this in Emacs built with it.
Myself I run the following command which will download a tarball of the
`master` branch, apply the fullscreen and sRGB patches, and build Emacs.app:
./build-emacs-for-osx --srgb
./build-emacs-for-osx
Or for example if you want to build the `EMACS_PRETEST_24_0_91` tag, run:
./build-emacs-for-osx --srgb EMACS_PRETEST_24_0_91
./build-emacs-for-osx EMACS_PRETEST_24_0_91
Resulting applications are saved to the `builds` directory in a bzip2
compressed tarball.

View File

@@ -46,7 +46,7 @@ end
def patches(opts = {})
p = []
if !opts[:no_fullscreen]
if opts[:fullscreen]
p << 'https://raw.github.com/gist/1012927'
end
@@ -63,18 +63,20 @@ end
#
def parse_options
options = {}
options = {:fullscreen => true, :srgb => true}
OptionParser.new do |opts|
opts.banner = "Usage: ./build-emacs-for-osx [options] [branch/tag/sha]"
opts.on('--srgb', "Include sRGB patch.") do
options[:srgb] = true
opts.on("--no-fullscreen", "Don't include fullscreen patch.") do
options[:fullscreen] = false
end
opts.on("--no-fullscreen", "Don't include fullscreen patch.") do
options[:no_fullscreen] = true
opts.on('--no-srgb', "Don't include sRGB patch.") do
options[:srgb] = false
end
opts.on("--srgb", "Option deprecated - sRGB patch is applied by default.")
end.parse!
options