Skip to content
This repository has been archived by the owner on Jan 16, 2025. It is now read-only.

Uberftp 2.8 #48447

Closed
wants to merge 4 commits into from
Closed

Uberftp 2.8 #48447

wants to merge 4 commits into from

Conversation

maarten-k
Copy link
Contributor

Update uberftp to version 2.8 based on work done by giffels (closed pull request #40848).

maarten-k added 2 commits January 25, 2016 16:49
Update uberftp to version 2.8 based on work done by giffels (closed pull request #40848).
The patch provided at JasonAlt/UberFTP#8 make the situation worse: another patch is needed to get configure running well and cause more obfuscation than the 2 replace commands. Replaced the single quotes in formula for double qoutes
Closes  #41277 and is based on work desribed in #40848.
The single quotes are replaced by double quotes. The patch JasonAlt/UberFTP#8 mentioned in #40848 solves the problem partly: there is still  another patch needed which obfuscate the formula even more then the replaces now.
when /gcc64dbg/ then "gcc64dbg"
when /gcc32dbg/ then "gcc32dbg"
end
#patches needed since location changed with globus-toolkit versions >= 6.0, patch to upstream not yet merged https://github.com/JasonAlt/UberFTP/pull/8, but solves not whole problem
Copy link
Member

Choose a reason for hiding this comment

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

Add space after # and wrap this line at 80 characters.

@apjanke
Copy link
Contributor

apjanke commented Jan 26, 2016

Looks good now. Could you squash this to a single commit so we have a tidy commit history?

@maarten-k
Copy link
Contributor Author

I will try to squash the commits today or tomorrow. Please bear with me: my experience with git is limited.

@MikeMcQuaid
Copy link
Member

@maarten-k I'll squash them for you.

@MikeMcQuaid
Copy link
Member

Thanks for your contribution to Homebrew! Without people like you submitting PRs we couldn't run this project. You rock!

@maarten-k
Copy link
Contributor Author

@MikeMcQuaid Thanks for squashing and for maintaining this awesome project!

@maarten-k
Copy link
Contributor Author

Closing, since the pull request is committed in master 03a12ec

@maarten-k maarten-k closed this Jan 26, 2016
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

uberftp fails to build on 10.10.3
3 participants