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

emulationstation: refactor #324444

Merged
merged 4 commits into from
Jul 28, 2024

Conversation

AndersonTorres
Copy link
Member

Found by af039eb#r143796136

Description of changes

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/)
  • 24.11 Release Notes (or backporting 23.11 and 24.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.

AndersonTorres referenced this pull request Jul 4, 2024
- finalAttrs design pattern
- remove eigen dependency
- strictDeps
- move resources to $out/share
- link resources to $out/bin
@ofborg ofborg bot requested a review from edwtjo July 4, 2024 01:20
@AndersonTorres AndersonTorres marked this pull request as ready for review July 4, 2024 01:43
@Aleksanaa Aleksanaa changed the title emulationstation: fix a ridiculous typo emulationstation: refactor Jul 6, 2024
@AndersonTorres
Copy link
Member Author

Since emulationstation needs an insecure freeimage (and I still did not figure out how to fix it), it will not be built by ofBorg.

@AndersonTorres AndersonTorres force-pushed the emulationstation branch 2 times, most recently from 43a611f to a75561d Compare July 12, 2024 13:51
@AndersonTorres AndersonTorres force-pushed the emulationstation branch 2 times, most recently from c2cb9b9 to f2c6a39 Compare July 24, 2024 03:07
Copy link
Contributor

@superherointj superherointj left a comment

Choose a reason for hiding this comment

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

I find this external sources pattern unusual. Since you are maintainer of package. I won't dispute your choices on this. Code-wise it works, yes. Just weird.

@superherointj
Copy link
Contributor

@edwtjo Do you have objections?

@thiagokokada thiagokokada merged commit 9d21d8a into NixOS:master Jul 28, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants