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

splash: update to v2.9.0 #4020

Merged
merged 2 commits into from Apr 15, 2019
Merged

splash: update to v2.9.0 #4020

merged 2 commits into from Apr 15, 2019

Conversation

danieljprice
Copy link
Contributor

  • update to version 2.9.0

Closes: https://trac.macports.org/attachment/ticket/58301/

Description

I submitted the trac ticket first, but then read that pull requests are preferred. Hence my first attempt, this one should be straightforward.

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.14.4
Xcode 10.2

Verification

Have you

  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@macportsbot
Copy link

Travis Build #5878 Failed.

Lint results
--->  Verifying Portfile for splash
--->  0 errors and 0 warnings found.

Port splash's dependencies fail on xcode10.2. Log
Port splash success on xcode9.4. Log
Port splash success on xcode8.3. Log

science/splash/Portfile Outdated Show resolved Hide resolved
@cjones051073
Copy link
Member

@danieljprice can you please address @mf2k ‘s comments, thanks ?

@macportsbot
Copy link

Travis Build #5916 Failed.

Lint results
--->  Verifying Portfile for splash
--->  0 errors and 0 warnings found.

Port splash success on xcode10.2. Log
Port splash success on xcode9.4. Log
Port splash's dependencies fail on xcode8.3. Log

@cjones051073
Copy link
Member

Thanks for adding size, but please follow our commit guidelines. You need to squash those two commits into one, and make sure the commit message follows the portname: explanation pattern.

@cjones051073
Copy link
Member

Ping @danieljprice . Can you address the comments please ?

@macportsbot
Copy link

Travis Build #5979 Failed.

Lint results
--->  Verifying Portfile for splash
--->  0 errors and 0 warnings found.

Port splash's dependencies fail on xcode10.2. Log
Port splash success on xcode9.4. Log
Port splash success on xcode8.3. Log

@macportsbot
Copy link

Travis Build #5980 Passed.

Lint results
--->  Verifying Portfile for splash
--->  0 errors and 0 warnings found.

Port splash success on xcode10.2. Log
Port splash success on xcode9.4. Log
Port splash success on xcode8.3. Log

@cjones051073
Copy link
Member

The PR still has two commits in it, one of which is now a merge commit, which means you did not properly rebase your changes before the last push. can you please address this ? This PR needs to show only one commit, your required changes, before it can be merged.

@cjones051073 cjones051073 merged commit cf44c2c into macports:master Apr 15, 2019
@cjones051073
Copy link
Member

I have manually squashed and merged this PR myself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants