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

amoebax, new recipe #670

Merged
merged 1 commit into from Jul 22, 2016
Merged

amoebax, new recipe #670

merged 1 commit into from Jul 22, 2016

Conversation

@Begasus
Copy link
Contributor

Begasus commented Jun 27, 2016

No description provided.

LICENSE="
GNU GPL v2
LALv1.2
"

This comment has been minimized.

Copy link
@fbrosson

fbrosson Jun 27, 2016

Member

Although PROVIDES*, REQUIRES*, BUILD_REQUIRES and BUILD_PREREQUIRES have newlines after the opening double quotes and before the closing double quotes, we do not use the same layout for COPYRIGHT and LICENSE. For these we use:

COPYRIGHT="2006-2007 Jordi Fita (Code)
    2006-2007 Safareig Creatiu (Graphics)
    2006-2007 Àlex Almarza (Music & Sound)
    2006-2007 Ferran Brugat (Web Page)"
LICENSE="GNU GPL v2
    LALv1.2"

or

COPYRIGHT="2006-2007 Jordi Fita (Code), Safareig Creatiu (Graphics), Àlex \
Almarza (Music & Sound), Ferran Brugat (Web Page)"
LICENSE="GNU GPL v2
    LALv1.2"

I think the second suggestion is slightly better, but I'm not sure about that.
BTW, I know a few recipes don't have the recommended layout, but for new ones it's better to stick to the recommendations ;-)

This comment has been minimized.

Copy link
@Begasus

Begasus Jun 28, 2016

Author Contributor

Ok, I'll take a look later and change the lines :)

@Begasus Begasus force-pushed the Begasus:amoebax branch from 43ee8de to eee8384 Jun 30, 2016
@Begasus
Copy link
Contributor Author

Begasus commented Jun 30, 2016

Changed the Copyright and Licenses section to meet the standard

@waddlesplash waddlesplash merged commit 6d131f6 into haikuports:master Jul 22, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Begasus Begasus deleted the Begasus:amoebax branch Dec 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.