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

Generic recipes: rework #161

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

Conversation

extrowerk
Copy link
Member

No description provided.

@extrowerk extrowerk changed the title Generic lib: libVersion, libVersionCompat, debuginfo, runconfigure Generic recipes: rework May 20, 2018
generic/generic_cmd-1.2.3.recipe Outdated Show resolved Hide resolved
generic/generic_haiku_app-1.2.3.recipe Outdated Show resolved Hide resolved
generic/generic_cmd-1.2.3.recipe Outdated Show resolved Hide resolved
generic/generic_cmd-1.2.3.recipe Show resolved Hide resolved
generic/generic_cmd-1.2.3.recipe Show resolved Hide resolved
@scottmc
Copy link
Member

scottmc commented Sep 14, 2018

Can someone wrap this one up? It'd be nice to have these in place in time for GCI

@extrowerk
Copy link
Member Author

Is there anything else what we want to include in the examples?

@@ -36,16 +36,48 @@ BUILD_PREREQUIRES="
cmd:gcc
"

defineDebugInfoPackage projectx \
"$appsDir"/ProjectX
Copy link
Member

Choose a reason for hiding this comment

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

Do we really need to quote $appsDir?

Copy link
Member Author

Choose a reason for hiding this comment

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

Better safe than sorry...

@@ -17,7 +17,7 @@ SOURCE_DIR="$portVersionedName"
PATCHES="projectx-$portVersion.patchset"
ADDITIONAL_FILES="projectx.rdef"

ARCHITECTURES="x86_gcc2 ?x86 ?x86_64"
ARCHITECTURES="?x86_gcc2 ?x86 ?x86_64"
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we remove x86 here (and in other places), there is no primary architecture for that anymore (afaik)

Copy link
Member

Choose a reason for hiding this comment

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

should update to show new use of 'all'

Comment on lines +47 to +49
GLOBAL_WRITABLE_FILES="
settings/projectx.conf keep-old
"
Copy link
Contributor

Choose a reason for hiding this comment

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

GLOBAL_WRITABLE_FILES should be used before PROVIDES

Comment on lines +42 to +44
GLOBAL_WRITABLE_FILES="
settings/projectx.rc keep-old # If there is any
"
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, needs to be declared before PROVIDES

Comment on lines +42 to +44
GLOBAL_WRITABLE_FILES="
settings/projectx.rc keep-old # If there is any
"
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here, should be declared before PROVIDES

Comment on lines +33 to +34
PYTHON_PACKAGES=(python python3)
PYTHON_VERSIONS=(2.7 3.6)
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe?

-PYTHON_PACKAGES=(python python3)
-PYTHON_VERSIONS=(2.7 3.6)
+PYTHON_PACKAGES=(python3.6 python3)
+PYTHON_VERSIONS=(3.6 3.7)

@@ -17,7 +17,7 @@ SOURCE_DIR="$portVersionedName"
PATCHES="projectx-$portVersion.patchset"
ADDITIONAL_FILES="projectx.rdef"

ARCHITECTURES="x86_gcc2 ?x86 ?x86_64"
ARCHITECTURES="?x86_gcc2 ?x86 ?x86_64"
Copy link
Member

Choose a reason for hiding this comment

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

should update to show new use of 'all'

PATCHES="projectx-$portVersion.patchset"
ADDITIONAL_FILES="projectx.rdef"

ARCHITECTURES="?x86_gcc2 ?x86 ?x86_64"
Copy link
Member

Choose a reason for hiding this comment

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

should update to show new use of 'all'

PATCHES="projectx-$portVersion.patchset"
ADDITIONAL_FILES="projectx.rdef"

ARCHITECTURES="?x86_gcc2 ?x86 ?x86_64"
Copy link
Member

Choose a reason for hiding this comment

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

should update to show new use of 'all'

PATCHES="projectx-$portVersion.patchset"
ADDITIONAL_FILES="projectx.rdef"

ARCHITECTURES="any"
Copy link
Member

Choose a reason for hiding this comment

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

should update to show new use of 'all'?

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.

7 participants