Skip to content

Conversation

@bengsparks
Copy link
Contributor

EB Garamond is currently made available by installing OTF files to the correct location.
By building the font from source using the GitHub repository instead of the bitbucket archive, the TTF and WOFF files also become available for installation.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

Copy link
Contributor

@niklaskorz niklaskorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure all commits are formatted using nixfmt-rfc-style as stated in CONTRIBUTING.md:

New files must be formatted by entering the nix-shell from the repository root and running nixfmt.

(not a new file in this case, but a previously already formatted file, hence the failing GA check)

@NixOSInfra NixOSInfra added the 12.first-time contribution This PR is the author's first one; please be gentle! label Jan 20, 2025
@github-actions github-actions bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jan 20, 2025
@nix-owners nix-owners bot requested review from relrod and rycee January 20, 2025 12:30
@bengsparks bengsparks force-pushed the eb-garamond-from-source branch 2 times, most recently from 477220d to c762189 Compare January 20, 2025 12:57
@niklaskorz
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 375307


x86_64-linux

❌ 1 package failed to build:
  • eb-garamond

aarch64-linux

❌ 1 package failed to build:
  • eb-garamond

x86_64-darwin

❌ 1 package failed to build:
  • eb-garamond

aarch64-darwin

❌ 1 package failed to build:
  • eb-garamond

@niklaskorz
Copy link
Contributor

error: build of '/nix/store/k3fsx9df8jkb93s2awsbh2fvvnlwrg54-source.drv' on 'ssh-ng://rosetta-builder' failed: hash mismatch in fixed-output derivation '/nix/store/k3fsx9df8jkb93s2awsbh2fvvnlwrg54-source.drv':
         specified: sha256-3eByiJ0gVpKXtKP8AH0P9G5vTbdvkmnfD4Dn2JQ7K54=
            got:    sha256-ajieKhTeH6yv2qiE2xqnHFoMS65//4ZKiccAlC2PXGQ=

@bengsparks bengsparks force-pushed the eb-garamond-from-source branch from c762189 to 1d95ce0 Compare January 20, 2025 13:24
@github-actions github-actions bot added 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Jan 20, 2025
@bengsparks bengsparks force-pushed the eb-garamond-from-source branch from 1d95ce0 to a4d19e7 Compare January 21, 2025 10:35
@bengsparks bengsparks requested a review from niklaskorz January 21, 2025 10:38
@bengsparks
Copy link
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 375307


aarch64-darwin

✅ 1 package built:
  • eb-garamond

@niklaskorz
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 375307


x86_64-linux

✅ 1 package built:
  • eb-garamond

@github-actions github-actions bot added the 8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` label Jan 21, 2025
@bengsparks bengsparks force-pushed the eb-garamond-from-source branch from 1cf4619 to ad97732 Compare January 21, 2025 11:18
Copy link
Contributor

@drupol drupol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trace: maintainer bengsparks (line 2845) should be listed after benesim, not after benhiemer (line 2795)

@niklaskorz
Copy link
Contributor

niklaskorz commented Jan 21, 2025

Please change the commit message from eb-garamond: add self as maintainer to eb-garamond: add bengsparks as maintainer (same for the maintainer-list commit)

@bengsparks bengsparks force-pushed the eb-garamond-from-source branch from ad97732 to e031d66 Compare January 21, 2025 11:22
@bengsparks bengsparks requested a review from drupol January 21, 2025 11:37
@drupol drupol merged commit fa569a3 into NixOS:master Jan 21, 2025
25 of 27 checks passed
@bengsparks bengsparks deleted the eb-garamond-from-source branch March 14, 2025 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: maintainer-list (update) This PR changes `maintainers/maintainer-list.nix` 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.first-time contribution This PR is the author's first one; please be gentle!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants