Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iosevka: 5.0.2 -> 7.0.4, iosevka-bin: 5.0.5 -> 7.0.4 #126664

Merged
merged 2 commits into from
Jun 13, 2021

Conversation

figgyc
Copy link
Contributor

@figgyc figgyc commented Jun 12, 2021

Motivation for this change

There has been a few new glyphs added to Iosevka since this package was last updated, I noticed as I was trying to compile a custom version and it didn't work with the current package.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • 21.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
  • Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

@marvin-mk2
Copy link

marvin-mk2 bot commented Jun 12, 2021

Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here.

@marvin-mk2 marvin-mk2 bot added marvin This PR was reviewed by Marvin, a discontinued bot: https://github.com/timokau/marvin-mk2 needs_reviewer (old Marvin label, do not use) labels Jun 12, 2021
@marvin-mk2 marvin-mk2 bot requested a review from kevincox June 12, 2021 12:35
@marvin-mk2 marvin-mk2 bot added awaiting_reviewer (old Marvin label, do not use) and removed needs_reviewer (old Marvin label, do not use) labels Jun 12, 2021
@ofborg ofborg bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 12, 2021
@SuperSandro2000
Copy link
Member

@figgyc please resolve the merge conflict.

@ofborg ofborg bot removed the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 12, 2021
@ofborg ofborg bot requested review from cstrahan and midchildan June 12, 2021 17:30
@r-rmcgibbo
Copy link

Result of nixpkgs-review pr 126664 at 410b187b run on aarch64-linux 1

1 package failed to build:
3 packages skipped due to time constraints:
  • iosevka
  • iosevka-bin
  • lumo
7 packages built successfully:
  • bat-extras.prettybat
  • discourse
  • nixos-install-tools
  • parity-ui
  • teleprompter
  • vscode-extensions.matklad.rust-analyzer
  • zerobin

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

@SuperSandro2000 SuperSandro2000 added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jun 12, 2021
@SuperSandro2000
Copy link
Member

sorry for the new merge conflict. Please hit someone up on matrix when you resolved it that we can merge this asap.

@ofborg ofborg bot added 10.rebuild-linux: 1-10 and removed 2.status: merge conflict This PR has merge conflicts with the target branch 10.rebuild-linux: 11-100 labels Jun 12, 2021
@figgyc
Copy link
Contributor Author

figgyc commented Jun 13, 2021

@SuperSandro2000 I've resolved the merge conflict, can you (or someone else) take another look? I did ask on matrix before I went to bed last night but seems like nobody did.

@kevincox kevincox merged commit b1f74dd into NixOS:master Jun 13, 2021
This was referenced Jun 29, 2021
@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
10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 12. first-time contribution This PR is the author's first one; please be gentle! awaiting_reviewer (old Marvin label, do not use) marvin This PR was reviewed by Marvin, a discontinued bot: https://github.com/timokau/marvin-mk2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants