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

Overpass: bump version #1099

Merged
merged 1 commit into from Apr 22, 2017
Merged

Conversation

@extrowerk
Copy link
Member

extrowerk commented Jan 21, 2017

It comes with 2 top level directory, that's why theese changes required.

REVISION="1"
SOURCE_URI="https://github.com/RedHatBrand/Overpass/releases/download/$portVersion/overpass-webfonts.zip"
CHECKSUM_SHA256="8dad24f7b12d359dad64b1ce0c4c57e833ac2523d7411db12904acb3f6f98658"
SOURCE_DIR="overpass-mono-webfont"

This comment has been minimized.

Copy link
@fbrosson

fbrosson Jan 21, 2017

Member

I think it would be worth inserting

SOURCE_FILENAME="overpass-webfonts-$portVersion.zip"

between CHECKSUM_SHA256 and SOURCE_DIR.
Otherwise HP will complain next time when a new archive becomes available.

It comes with 2 top level directory, that's why theese changes required.

Regarding SOURCE_URI_2, I'm pretty sure you can workaround this (to avoid having to define SOURCE_URI_2) by setting

SOURCE_DIR=""
Copy link
Contributor

korli left a comment

@miqlas any update?

@korli
Copy link
Contributor

korli commented Mar 21, 2017

@fbrosson what's up? I understand you have probably no time at the moment. Wondering whether something went wrong.

@fbrosson
Copy link
Member

fbrosson commented Mar 21, 2017

Don't worry. Nothing went wrong. It's just that I decided to switch to something else.

@Begasus
Copy link
Contributor

Begasus commented Mar 22, 2017

Hope you'll stick around for a while @fbrosson .... I've learned a lot from you, and I'm sure others did also! Anyway ... best of luck with the new adventure!

@waddlesplash
Copy link
Member

waddlesplash commented Apr 22, 2017

eh, SOURCE_FILENAME change isn't really needed.

@waddlesplash waddlesplash merged commit 6678ec5 into haikuports:master Apr 22, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
CodeforEvolution pushed a commit to CodeforEvolution/haikuports that referenced this pull request May 6, 2017
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

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