Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Added beta to linklint formula #31977

Closed
wants to merge 5 commits into from
Closed

Conversation

keithws
Copy link
Contributor

@keithws keithws commented Aug 30, 2014

Addressed issues in #31079 after it was closed. I updated the test to write a file and run it through the program like the cmake test does. I also replace the patch with a simple "inreplace" line since the upstream is abandoned.

@MikeMcQuaid
Copy link
Member

Can you try and submit this upstream anyway, thanks.


# fix version number reported by linklint -version in beta
# note, upstream is abandoned, so inreplace instead of patch
inreplace "linklint-#{version}", "2.3.1", "#{version}" if build.devel?
Copy link
Member

Choose a reason for hiding this comment

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

"#{version}" should be version

@jacknagel
Copy link
Contributor

ping

@keithws
Copy link
Contributor Author

keithws commented Sep 7, 2014

Fixed version and rebased.

@keithws
Copy link
Contributor Author

keithws commented Sep 7, 2014

I emailed the original author with a patch to correct the version number.

@jacknagel jacknagel closed this in dfe114d Sep 8, 2014
@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 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.

3 participants