X-Git-Url: http://git.hungrycats.org/cgi-bin/gitweb.cgi?p=xscreensaver;a=blobdiff_plain;f=OSX%2Fupdate-info-plist.pl;h=0d933cb7f6d69fa081a03f02dfc1f04bb2b759f9;hp=998c452399efc9d5741a9159614967574bd5681e;hb=019de959b265701cd0c3fccbb61f2b69f06bf9ee;hpb=f8cf5ac7b2f53510f80a0eaf286a25298be17bfe diff --git a/OSX/update-info-plist.pl b/OSX/update-info-plist.pl index 998c4523..0d933cb7 100755 --- a/OSX/update-info-plist.pl +++ b/OSX/update-info-plist.pl @@ -1,5 +1,5 @@ #!/usr/bin/perl -w -# Copyright © 2006-2012 Jamie Zawinski +# Copyright © 2006-2013 Jamie Zawinski # # Permission to use, copy, modify, distribute, and sell this software and its # documentation for any purpose is hereby granted without fee, provided that @@ -21,16 +21,40 @@ require 5; #use diagnostics; # Fails on some MacOS 10.5 systems use strict; +use IPC::Open3; +use IO::Uncompress::Gunzip qw(gunzip $GunzipError); +use IO::Compress::Gzip qw(gzip $GzipError); -my $progname = $0; $progname =~ s@.*/@@g; -my $version = q{ $Revision: 1.21 $ }; $version =~ s/^[^0-9]+([0-9.]+).*$/$1/; +my ($exec_dir, $progname) = ($0 =~ m@^(.*?)/([^/]+)$@); + +my $version = q{ $Revision: 1.28 $ }; $version =~ s/^[^0-9]+([0-9.]+).*$/$1/; $ENV{PATH} = "/usr/local/bin:$ENV{PATH}"; # for seticon +my $thumbdir = $ENV{HOME} . '/www/xscreensaver/screenshots/'; + + my $verbose = 1; -sub read_info_plist($); +sub convert_plist($$) { + my ($data, $to_binary_p) = @_; + my $is_binary_p = ($data =~ m/^bplist/s); + if ($data && (!$is_binary_p) != (!$to_binary_p)) { + print STDERR "$progname: converting plist\n" if ($verbose > 2); + my $which = ($to_binary_p ? 'binary1' : 'xml1'); + my $cmd = "plutil -convert $which -s -o - -"; + my $pid = open3 (my $in, my $out, undef, $cmd) || error ("pipe: $cmd: $!"); + print $in $data; + close $in; + local $/ = undef; # read entire file + $data = <$out>; + close $out; + } + return $data; +} + + sub read_info_plist($) { my ($app_dir) = @_; my $file = "$app_dir/Contents/Info.plist"; @@ -43,16 +67,12 @@ sub read_info_plist($) { } else { error ("$file: $!"); } + print STDERR "$progname: read $file\n" if ($verbose > 2); local $/ = undef; # read entire file my $body = <$in>; close $in; - if ($body =~ m/^bplist/s) { - print STDERR "$progname: converting binary plist file: $file\n"; - system ("plutil", "-convert", "xml1", $file); - return read_info_plist ($app_dir); - } - + $body = convert_plist ($body, 0); # convert to xml plist return ($file, $body); } @@ -65,6 +85,7 @@ sub read_saver_xml($) { return () if ($name eq 'XScreenSaver'); return () if ($name eq 'SaverTester'); + return () if ($name eq 'XScreenSaverUpdater'); my $file = "$app_dir/Contents/Resources/" . lc($name) . ".xml"; my $file2 = "$app_dir/" . lc($name) . ".xml"; @@ -78,16 +99,22 @@ sub read_saver_xml($) { } else { error ("$file: $!"); } + print STDERR "$progname: read $file\n" if ($verbose > 2); local $/ = undef; # read entire file my $body = <$in>; close $in; - return ($file, $body); + + # Uncompress the XML if it is compressed. + my $body2 = ''; + gunzip (\$body, \$body2) || error ("$app_dir: xml gunzip: $GunzipError"); + my $was_compressed_p = ($body ne $body2); + return ($file, $body2, $was_compressed_p); } sub update_saver_xml($$) { my ($app_dir, $vers) = @_; - my ($filename, $body) = read_saver_xml ($app_dir); + my ($filename, $body, $was_compressed_p) = read_saver_xml ($app_dir); my $obody = $body; return () unless defined ($filename); @@ -152,7 +179,7 @@ sub update_saver_xml($$) { "\n" . "From the XScreenSaver collection: " . "http://www.jwz.org/xscreensaver/\n" . - "Copyright \251 $year by $authors.\n"); + "Copyright \302\251 $year by $authors.\n"); my $desc2 = ("$name $vers,\n" . # Info.plist "\302\251 $year $authors.\n" . @@ -171,13 +198,23 @@ sub update_saver_xml($$) { $body =~ s@(<_description>)(.*?)()@$1$desc1$3@s; - if ($obody eq $body) { + # NSXMLParser doesn't seem to work properly on Latin1 XML documents, + # so we convert these to UTF8 when embedding them in the .saver bundle. + $body =~ s@encoding="ISO-8859-1"@encoding="UTF-8"@gsi; + + if ($obody eq $body && $was_compressed_p) { print STDERR "$progname: $filename: unchanged\n" if ($verbose > 1); } else { + + # Gzip the XML. + my $body2 = ''; + gzip (\$body, \$body2) || error ("$app_dir: xml gzip: $GzipError"); + $body = $body2; + my $file_tmp = "$filename.tmp"; - open(OUT, ">$file_tmp") || error ("$file_tmp: $!"); - print OUT $body || error ("$file_tmp: $!"); - close OUT || error ("$file_tmp: $!"); + open (my $out, '>:raw', $file_tmp) || error ("$file_tmp: $!"); + print $out $body || error ("$file_tmp: $!"); + close $out || error ("$file_tmp: $!"); if (!rename ("$file_tmp", "$filename")) { unlink "$file_tmp"; @@ -190,6 +227,40 @@ sub update_saver_xml($$) { } +sub compress_all_xml_files($) { + my ($dir) = @_; + opendir (my $dirp, $dir) || error ("$dir: $!"); + my @files = readdir ($dirp); + closedir $dirp; + foreach my $f (sort @files) { + next unless ($f =~ m/\.xml$/si); + my $filename = "$dir/$f"; + open (my $in, '<', $filename) || error ("$filename: $!"); + print STDERR "$progname: read $filename\n" if ($verbose > 2); + local $/ = undef; # read entire file + my $body = <$in>; + close $in; + + if ($body =~ m/^<\?xml/s) { + my $body2 = ''; + gzip (\$body, \$body2) || error ("$filename: xml gzip: $GzipError"); + $body = $body2; + my $file_tmp = "$filename.tmp"; + open (my $out, '>:raw', $file_tmp) || error ("$file_tmp: $!"); + print $out $body || error ("$file_tmp: $!"); + close $out || error ("$file_tmp: $!"); + + if (!rename ("$file_tmp", "$filename")) { + unlink "$file_tmp"; + error ("mv \"$file_tmp\" \"$filename\": $!"); + } + print STDERR "$progname: compressed $filename\n" if ($verbose); + } elsif ($verbose > 2) { + print STDERR "$filename: already compressed\n"; + } + } +} + sub set_plist_key($$$$) { my ($filename, $body, $key, $val) = @_; @@ -228,6 +299,21 @@ sub set_icon($) { } +sub set_thumb($) { + my ($app_dir) = @_; + + return unless ($app_dir =~ m@\.saver/?$@s); + + my @cmd = ("$exec_dir/update-thumbnail.pl", $thumbdir, $app_dir); + push @cmd, "-" . ("v" x $verbose) if ($verbose); + print STDERR "$progname: exec: " . join(' ', @cmd) . "\n" + if ($verbose > 1); + system (@cmd); + my $exit = $? >> 8; + exit ($exit) if $exit; +} + + sub update($) { my ($app_dir) = @_; @@ -244,7 +330,12 @@ sub update($) { my $vers = $1; my ($ignore, $info_str) = update_saver_xml ($app_dir, $vers); - if (! defined($info_str)) { + # No, don't do this -- the iOS version reads the XML file in a few + # different places, and most of those places don't understand gzip. + + if ($app_name eq 'XScreenSaver') { + compress_all_xml_files ($app_dir); + } elsif (! defined($info_str)) { print STDERR "$progname: $filename: no XML file\n" if ($verbose > 1); } else { @@ -268,10 +359,11 @@ sub update($) { if ($oplist eq $plist) { print STDERR "$progname: $filename: unchanged\n" if ($verbose > 1); } else { + $plist = convert_plist ($plist, 1); # convert to binary plist my $file_tmp = "$filename.tmp"; - open(OUT, ">$file_tmp") || error ("$file_tmp: $!"); - print OUT $plist || error ("$file_tmp: $!"); - close OUT || error ("$file_tmp: $!"); + open (my $out, '>:raw', $file_tmp) || error ("$file_tmp: $!"); + print $out $plist || error ("$file_tmp: $!"); + close $out || error ("$file_tmp: $!"); if (!rename ("$file_tmp", "$filename")) { unlink "$file_tmp"; @@ -282,6 +374,7 @@ sub update($) { } set_icon ($app_dir); + set_thumb ($app_dir); }