Content-Length: 297040 | pFad | http://github.com/searxng/searxng/pull/4170

90 [feat] Added shell.nix (Nix development support) by Denperidge · Pull Request #4170 · searxng/searxng · GitHub
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

[feat] Added shell.nix (Nix development support) #4170

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Denperidge
Copy link
Contributor

What does this PR do?

Add a shell.nix file. This is not for use in the Nix version of searchx, but instead allows developers to get a compatible development environment in one command.

Why is this change important?

Some developers are NixOS based, which cannot run arbitrary executables (e.g. the nvm-based node) by default. The nix-shell wraps it in an FHS environment, which fixes that thanks to Nix magic I do not understand.

How to test this PR locally?

On a device with nix installed, navigate to the repository and run make test. This will probably not work. Then, run nix-shell and try make test again. It should work!

@return42
Copy link
Member

@SuperSandro2000 can you review this PR? .. thanks 👍

Done!
*/

(pkgs.buildFHSEnv {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why didn't you use mkShell?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Didn't let me run the node nvm stuff from the makefile otherwise! I'm on NixOS 24.11 but in case this was a my-config-issue then we can switch it

Copy link
Contributor

Choose a reason for hiding this comment

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

Do you still have the error?

shell.nix Outdated Show resolved Hide resolved
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
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.

3 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/searxng/searxng/pull/4170

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy