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

Add NetBSD support #177

Merged
merged 4 commits into from Dec 7, 2023
Merged

Add NetBSD support #177

merged 4 commits into from Dec 7, 2023

Conversation

FGasper
Copy link
Contributor

@FGasper FGasper commented Dec 7, 2023

No description provided.

@FGasper FGasper marked this pull request as draft December 7, 2023 04:24
@saghul
Copy link
Contributor

saghul commented Dec 7, 2023

Looks like the way to get the number of cpus is different on netbsd...

@saghul
Copy link
Contributor

saghul commented Dec 7, 2023

Or you can wait until #178 lands and it should take care of it...

Comment on lines 429 to 434
/usr/sbin/pkg_add cmake
run: |
cmake -B build
cmake --build build -j $(sysctl -n hw.ncpu)
./build/qjs -qd
make test
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
/usr/sbin/pkg_add cmake
run: |
cmake -B build
cmake --build build -j $(sysctl -n hw.ncpu)
./build/qjs -qd
make test
/usr/sbin/pkg_add cmake gmake
run: |
cmake -B build
cmake --build build -j $(sysctl -n hw.ncpu)
./build/qjs -qd
gmake test

@saghul
Copy link
Contributor

saghul commented Dec 7, 2023

If you rebase now, and install gmake, it should just work.

@FGasper FGasper marked this pull request as ready for review December 7, 2023 14:26
@FGasper
Copy link
Contributor Author

FGasper commented Dec 7, 2023

@bnoordhuis Should be good to go.

@saghul saghul merged commit 9de1526 into quickjs-ng:master Dec 7, 2023
31 checks passed
@FGasper FGasper deleted the netbsd branch December 7, 2023 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants