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.

Sign up for GitHub

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

Jump to bottom

Remove incompatible OS X switches in Test.py #1757

Closed
wants to merge 1 commit into from
Closed

Remove incompatible OS X switches in Test.py #1757

wants to merge 1 commit into from

Conversation

miguelportilla
Copy link
Contributor

No description provided.

@HowardHinnant
Copy link
Contributor

I needed to set the execute permission flags on Test.py: https://github.com/HowardHinnant/rippled/commits/test_script
After that, everything built fine.

@miguelportilla
Copy link
Contributor Author

@HowardHinnant 100755 → 100644 Thanks, will fix.

@HowardHinnant
Copy link
Contributor

👍

@scottschurr
Copy link
Collaborator

I had the same file permission issue as @HowardHinnant, so I invoked with
python Builds/Test.py -a
It took a while 😄 but it all worked. 👍

@miguelportilla miguelportilla added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Jun 27, 2016
@miguelportilla miguelportilla deleted the test_script branch June 29, 2016 21:28
@khancode khancode mentioned this pull request Jun 30, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@HowardHinnant HowardHinnant

@scottschurr scottschurr

Labels
Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@miguelportilla @HowardHinnant @scottschurr @vinniefalco