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

added scons-3.1.1 receipt #4437

Merged
merged 3 commits into from Dec 12, 2019
Merged

added scons-3.1.1 receipt #4437

merged 3 commits into from Dec 12, 2019

Conversation

LordOfDragons
Copy link
Contributor

No description provided.

@extrowerk
Copy link
Member

Warning: POLICY ERROR: no matching provides "cmd:scons_3.1.1.bat" for "bin/scons-3.1.1.bat"
Warning: POLICY ERROR: no matching provides "cmd:scons.bat" for "bin/scons.bat"
Warning: Error: packaging policy violation(s) in strict mode
Error: Build has failed - stopping.

@extrowerk
Copy link
Member

REVISION should be 1 as it is a new version.

@extrowerk
Copy link
Member

Here is my version, it should work ok: http://0x0.st/zUme.recipe
I cleaned up the -INSTALL section too, no idea, why was it so complicated.

I don't like the idea to make it depend on python3.6, the current python3 version is 3.7.5, and i think we should use that instead.

@LordOfDragons
Copy link
Contributor Author

Here is my version, it should work ok: http://0x0.st/zUme.recipe
I cleaned up the -INSTALL section too, no idea, why was it so complicated.

I don't like the idea to make it depend on python3.6, the current python3 version is 3.7.5, and i think we should use that instead.

Can the recipe point to "python3" or is it required to point to a specific version like "python3.7.5"? SCons 3.1.1 is compatible with python 3.5+. Can I specify something like a minimum version?

@mmuman
Copy link
Member

mmuman commented Dec 10, 2019

Some grep shows:
sci-mathematics/ginac/ginac-1.6.7.recipe: cmd:python >= 2.7

@extrowerk
Copy link
Member

extrowerk commented Dec 10, 2019 via email

- cleaned up install section
- changed python3.6 to python3
@LordOfDragons
Copy link
Contributor Author

Commited the changes. Is it better now?

@korli korli merged commit fabd12b into haikuports:master Dec 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants