Skip to content

Add a patch to allow true-name to build with current template-haskell.#208428

Merged
cdepillabout merged 1 commit intoNixOS:haskell-updatesfrom
nuttycom:haskell-unbreak_true_name
Jan 2, 2023
Merged

Add a patch to allow true-name to build with current template-haskell.#208428
cdepillabout merged 1 commit intoNixOS:haskell-updatesfrom
nuttycom:haskell-unbreak_true_name

Conversation

@nuttycom
Copy link
Contributor

Description of changes

This patches the Haskell true-name package to build with current ghc versions available from nixpkgs.
The patch includes the updates included in the 0.1.0.4 version of true-name (which has never had a Hackage release) and also applies https://github.com/liyang/true-name/pull/4.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 23.05 Release Notes (or backporting 22.11 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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@github-actions github-actions bot added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Dec 30, 2022
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Dec 30, 2022
@nuttycom nuttycom marked this pull request as draft January 1, 2023 19:27
@nuttycom
Copy link
Contributor Author

nuttycom commented Jan 1, 2023

Moved to draft in order to pull the patch via fetchpatch

@nuttycom nuttycom force-pushed the haskell-unbreak_true_name branch from 4b909a4 to 6d54eb7 Compare January 1, 2023 23:03
@nuttycom nuttycom marked this pull request as ready for review January 1, 2023 23:12
@nuttycom
Copy link
Contributor Author

nuttycom commented Jan 1, 2023

Rebased & ready for review.

@cdepillabout
Copy link
Member

Thanks, this looks good!

One more quick thing though. Commit messages in Nixpkgs are supposed to have a specific format:

https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md

So the commit in this PR should maybe look like:

haskellPackages.true-name: Add a patch to allow building with current template-haskell

Other than that, this looks good. Sorry for the back-and-forth!

… template-haskell.

This updates the `true-name` package to include changes up to version
0.1.0.4, as well as compilation failures when building with
template-haskell version 2.16 and higher.
@nuttycom nuttycom force-pushed the haskell-unbreak_true_name branch from 6d54eb7 to 2d3c98f Compare January 2, 2023 05:11
@nuttycom
Copy link
Contributor Author

nuttycom commented Jan 2, 2023

force-pushed to fix the commit message, thanks!

@cdepillabout cdepillabout merged commit c99e04f into NixOS:haskell-updates Jan 2, 2023
@nuttycom nuttycom deleted the haskell-unbreak_true_name branch January 2, 2023 16:32
@Janik-Haag Janik-Haag added the 12.first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: haskell General-purpose, statically typed, purely functional programming language 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages 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.

3 participants