chore(master): release 0.6.53 (#123)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

.github/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.6.50"
+  ".": "0.6.53"
 }

CHANGELOG.md

@@ -1,5 +1,26 @@
 # Changelog
 
+## [0.6.53](https://github.com/jimeh/build-emacs-for-macos/compare/v0.6.52...v0.6.53) (2024-12-01)
+
+
+### Bug Fixes
+
+* **help:** correct formatting of help text output ([db72381](https://github.com/jimeh/build-emacs-for-macos/commit/db723817bf6c0ac85da1790a1d50fbea774cc0c0))
+
+## [0.6.52](https://github.com/jimeh/build-emacs-for-macos/compare/v0.6.51...v0.6.52) (2024-11-30)
+
+
+### Bug Fixes
+
+* **nix/flake:** rename flake-package-versions.txt to flake.pkgs ([d7723ee](https://github.com/jimeh/build-emacs-for-macos/commit/d7723ee766574b6597997de7c54fb7ed7f37965c))
+
+## [0.6.51](https://github.com/jimeh/build-emacs-for-macos/compare/v0.6.50...v0.6.51) (2024-11-28)
+
+
+### Bug Fixes
+
+* **build/embed:** include all Emacs C sources and related files in the Emacs.app bundle ([#120](https://github.com/jimeh/build-emacs-for-macos/issues/120)) ([ccb4f3f](https://github.com/jimeh/build-emacs-for-macos/commit/ccb4f3f438652c7ae98c202b8afed8861f40eeec))
+
 ## [0.6.50](https://github.com/jimeh/build-emacs-for-macos/compare/v0.6.49...v0.6.50) (2024-11-25)
 
 

Makefile

@@ -137,7 +137,7 @@ gen:
 .PHONY: nix-flake-update
 nix-flake-update:
 	nix flake update \
-	&& $(MAKE) flake-package-versions.txt
+	&& $(MAKE) flake.pkgs
 
 .SILENT: flake-package-versions
 flake-package-versions:
@@ -146,8 +146,8 @@ flake-package-versions:
 			$$(echo $$PATH | tr ":" "\n" | grep "/nix/store" | sort -u) \
 			| jq -r ".[].name" | sort -u'
 
-flake-package-versions.txt: flake.nix flake.lock
-	$(MAKE) flake-package-versions > flake-package-versions.txt
+flake.pkgs: flake.nix flake.lock
+	$(MAKE) flake-package-versions > "$@"
 
 #
 # Dependencies

README.md

@@ -73,23 +73,26 @@ make bootstrap
 
 ## Status
 
-As of writing (2023-11-20) it works for me on my machine and for the nightly
+As of writing (2024-11-30) it works for me on my machine and for the nightly
 builds in [jimeh/emacs-builds](https://github.com/jimeh/emacs-builds). Your luck
 may vary.
 
 I have successfully built:
 
-- `emacs-29.1` release tag.
-- `master` branch (Emacs 30.x).
+- `emacs-29.4` release tag.
+- `emacs-30.0.92` pretest tag.
+- `master` branch (Emacs 31.x).
 
 For reference, my machine is:
 
-- 14-inch MacBook Pro (2023), Apple M3 Pro (11-cores)
-- macOS Sonoma 14.1.1 (23B2082)
-- Xcode 15.0.1 (15A507)
+- 14-inch MacBook Pro (2023), Apple M3 Max (16-cores)
+- macOS Sonoma 15.1.1 (24B91)
+- Xcode 16.1 (16B40)
 
-Nightly builds are built with GitHub Actions on GitHub-hosted runners, using
-`macos-12` for Intel builds, and `macos-13-xlarge` for Apple Silicon builds.
+The [nightly builds](https://github.com/jimeh/emacs-builds) are built with
+GitHub Actions on GitHub-hosted runners, using `macos-13` for Intel builds, and
+`macos-14` for Apple Silicon builds. The build environment is managed with Nix,
+and targets the macOS 11 SDK.
 
 ## Usage
 
@@ -105,6 +108,11 @@ Or you can run the build script via `nix develop`:
 nix develop --command ./build-emacs-for-macos --help
 ```
 
+The Nix environment defaults to targeting the macOS 11 SDK, which makes Emacs
+builds compatible with macOS 11.3 or later. You can easily target later macOS
+SDKs. Versions 11 to 15 are available. For example, to target the macOS 12 SDK,
+run `nix develop .#macos12`
+
 ### Homebrew
 
 Run `make boostrap` to ensure all Ruby and Homebrew dependencies are installed.
@@ -126,7 +134,10 @@ Options:
         --[no-]xwidgets              Enable/disable XWidgets if supported (default: enabled)
         --[no-]tree-sitter           Enable/disable tree-sitter if supported (default: enabled)
         --[no-]native-comp           Enable/disable native-comp (default: enabled if supported)
-        --[no-]native-march          Enable/disable -march=native CFLAG(default: disabled)
+        --optimize                   Shorthand for --native-march --native-mtune --fomit-frame-pointer (default: disabled)
+        --[no-]native-march          Enable/disable -march=native CFLAG (default: disabled)
+        --[no-]native-mtune          Enable/disable -mtune=native CFLAG (default: disabled)
+        --[no-]fomit-frame-pointer   Enable/disable -fomit-frame-pointer CFLAG (default: disabled)
         --[no-]native-full-aot       Enable/disable NATIVE_FULL_AOT / Ahead of Time compilation (default: disabled)
         --[no-]relink-eln-files      Enable/disable re-linking shared libraries in bundled *.eln files (default: enabled)
         --[no-]rsvg                  Enable/disable SVG image support via librsvg (default: enabled)
@@ -168,10 +179,10 @@ as of writing) and build Emacs.app from it:
 ./build-emacs-for-macos
 ```
 
-To build the stable `emacs-29.1` release git tag run:
+To build the stable `emacs-29.4` release git tag run:
 
 ```
-./build-emacs-for-macos emacs-29.1
+./build-emacs-for-macos emacs-29.4
 ```
 
 All sources as downloaded as tarballs from the

build-emacs-for-macos

@@ -409,6 +409,10 @@ class Build
 
     patches.each { |patch| apply_patch(patch, target) }
 
+    # Keep a copy of src after patches have been applied. This will be used to
+    # embed C sources into the output Emacs.app bundle.
+    cmd('cp', '-a', File.join(target, 'src'), File.join(target, 'src.orig'))
+
     target
   end
 
@@ -1136,8 +1140,12 @@ class AbstractEmbedder
     @app = app
   end
 
-  def relative_path(path)
-    Pathname.new(path).relative_path_from(Pathname.new(app)).to_s
+  def relative_path(base, path)
+    Pathname.new(path).relative_path_from(Pathname.new(base)).to_s
+  end
+
+  def relative_app_path(path)
+    relative_path(app, path)
   end
 
   def invocation_dir
@@ -1207,16 +1215,18 @@ class CSourcesEmbedder < AbstractEmbedder
   def embed
     info 'Bundling C source files into Emacs.app for documentation purposes...'
 
+    orig_src_dir = File.join(source_dir, 'src.orig')
     src_dir = File.join(source_dir, 'src')
     target_dir = File.join(resources_dir, 'src')
-    debug "Copying *.c and *.h files from '#{src_dir}' " \
-          "to: #{relative_path(target_dir)}"
 
-    Dir[File.join(src_dir, '**', '*.{c,h}')].each do |f|
-      rel = f[src_dir.size + 1..-1]
-      target = File.join(target_dir, rel)
-      FileUtils.mkdir_p(File.dirname(target))
-      cmd('cp', '-pRL', f, target)
+    debug "Copying Emacs C sources to #{relative_app_path(target_dir)}"
+
+    if File.exist?(orig_src_dir)
+      copy_sources(orig_src_dir, target_dir)
+    else
+      copy_sources(
+        src_dir, target_dir, File.join('**', '*.{awk,c,cc,h,in,m,mk}')
+      )
     end
 
     if File.exist?(site_start_el_file) &&
@@ -1224,13 +1234,25 @@ class CSourcesEmbedder < AbstractEmbedder
       return
     end
 
-    debug "Patching '#{relative_path(site_start_el_file)}' to allow Emacs to " \
-          'find bundled C sources'
+    debug "Patching '#{relative_app_path(site_start_el_file)}' to allow " \
+          'Emacs to find bundled C sources'
     File.open(site_start_el_file, 'a') { |f| f.puts("\n#{PATH_PATCH}") }
   end
 
   private
 
+  def copy_sources(src_dir, target_dir, pattern = File.join('**', '*'))
+    Dir[File.join(src_dir, pattern)].each do |f|
+      next if File.directory?(f) ||
+              File.basename(f).downcase.start_with?('changelog')
+
+      rel = relative_path(src_dir, f)
+      target = File.join(target_dir, rel)
+      FileUtils.mkdir_p(File.dirname(target))
+      run_cmd('cp', '-pRL', f, target)
+    end
+  end
+
   def site_start_el_file
     @site_start_el_file ||= File.join(resources_dir, 'lisp', 'site-start.el')
   end
@@ -1256,10 +1278,9 @@ class LibEmbedder < AbstractEmbedder
     binary ||= bin
 
     FileUtils.cd(File.dirname(app)) do
-      rel_path = Pathname.new(lib_dir).relative_path_from(
-        Pathname.new(File.dirname(binary))
-      ).to_s
-      rpath = File.join('@executable_path', rel_path)
+      rpath = File.join(
+        '@executable_path', relative_path(File.dirname(binary), lib_dir)
+      )
 
       copy, relink = build_bundle_plan(binary)
 
@@ -1300,7 +1321,7 @@ class LibEmbedder < AbstractEmbedder
 
     return if mf.rpaths.include?(rpath)
 
-    debug "Setting rpath for '#{relative_path(macho_file)}' to: #{rpath}"
+    debug "Setting rpath for '#{relative_app_path(macho_file)}' to: #{rpath}"
     mf.add_rpath(rpath)
     while_writable(macho_file) { mf.write! }
   end
@@ -1330,7 +1351,7 @@ class LibEmbedder < AbstractEmbedder
 
     if macho_file.start_with?(app)
       debug 'Calculating bundling instructions for: ' \
-            "#{relative_path(macho_file)}"
+            "#{relative_app_path(macho_file)}"
     else
       debug "Calculating bundling instructions for: #{macho_file}"
     end
@@ -1413,7 +1434,8 @@ class LibEmbedder < AbstractEmbedder
 
       next if File.exist?(target)
 
-      debug "Copying '#{source}' to: '#{relative_path(target)}' ('#{dylib_id}')"
+      debug "Copying '#{source}' to: " \
+            "'#{relative_app_path(target)}' ('#{dylib_id}')"
       FileUtils.mkdir_p(File.dirname(target))
       cmd('cp', '-pRL', source, target)
 
@@ -1434,7 +1456,7 @@ class LibEmbedder < AbstractEmbedder
 
     relink_files = relink.group_by { |r| r[:target_file] }
     relink_files.each do |target_file, relinks|
-      debug "Changing linked dylibs in: '#{relative_path(target_file)}'"
+      debug "Changing linked dylibs in: '#{relative_app_path(target_file)}'"
       mf = MachO.open(target_file)
       changed = false
 
@@ -1556,7 +1578,7 @@ class GccLibEmbedder < AbstractEmbedder
       rpaths = mf.rpaths.reject { |r| r == '@loader_path' }
       next if rpaths.none?
 
-      debug "Tidying up rpaths from: #{relative_path(file_path)}"
+      debug "Tidying up rpaths from: #{relative_app_path(file_path)}"
       rpaths.each { |r| mf.delete_rpath(r) }
       mf.write!
     end
@@ -1586,10 +1608,6 @@ class GccLibEmbedder < AbstractEmbedder
     gcc_info.darwin_lib_dir
   end
 
-  def relative_dir(path, root)
-    Pathname.new(path).relative_path_from(Pathname.new(root)).to_s
-  end
-
   def site_start_el_file
     @site_start_el_file ||= File.join(resources_dir, 'lisp', 'site-start.el')
   end
@@ -1676,17 +1694,17 @@ class GccInfo
 
   def app_bundle_target_lib_dir
     @app_bundle_target_lib_dir ||=
-      relative_dir(
-        File.join(embedder.lib_dir, target_lib_dir),
-        embedder.invocation_dir
+      relative_path(
+        embedder.invocation_dir,
+        File.join(embedder.lib_dir, target_lib_dir)
       )
   end
 
   def app_bundle_target_darwin_lib_dir
     @app_bundle_target_darwin_lib_dir ||=
-      relative_dir(
-        File.join(embedder.lib_dir, sanitized_target_darwin_lib_dir),
-        embedder.invocation_dir
+      relative_path(
+        embedder.invocation_dir,
+        File.join(embedder.lib_dir, sanitized_target_darwin_lib_dir)
       )
   end
 
@@ -1761,8 +1779,8 @@ class GccInfo
     @embedder ||= AbstractEmbedder.new(Dir.mktmpdir(%w[Emacs .app]))
   end
 
-  def relative_dir(path, root)
-    Pathname.new(path).relative_path_from(Pathname.new(root)).to_s
+  def relative_path(base, path)
+    Pathname.new(path).relative_path_from(Pathname.new(base)).to_s
   end
 end
 
@@ -1849,7 +1867,7 @@ if __FILE__ == $PROGRAM_NAME
 
     opts.on(
       '--optimize',
-      'Shorthand for --native-march --native-mtune --fomit-frame-pointer' \
+      'Shorthand for --native-march --native-mtune --fomit-frame-pointer ' \
       '(default: disabled)'
     ) do
       cli_options[:native_march] = true
@@ -1859,19 +1877,19 @@ if __FILE__ == $PROGRAM_NAME
 
     opts.on(
       '--[no-]native-march',
-      'Enable/disable -march=native CFLAG' \
+      'Enable/disable -march=native CFLAG ' \
       '(default: disabled)'
     ) { |v| cli_options[:native_march] = v }
 
     opts.on(
       '--[no-]native-mtune',
-      'Enable/disable -mtune=native CFLAG' \
+      'Enable/disable -mtune=native CFLAG ' \
       '(default: disabled)'
     ) { |v| cli_options[:native_mtune] = v }
 
     opts.on(
       '--[no-]fomit-frame-pointer',
-      'Enable/disable -fomit-frame-pointer CFLAG' \
+      'Enable/disable -fomit-frame-pointer CFLAG ' \
       '(default: disabled)'
     ) { |v| cli_options[:fomit_frame_pointer] = v }