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

Fixes #11817: Scan Tailor now builds with Clang #27919

Closed
wants to merge 1 commit into from

Conversation

muellermartin
Copy link
Contributor

Updates scantailor to 0.9.11.1 (or scantailor-enhanced-20140214 for --devel) and fixes building with Clang

cause "calling a private constructor of class 'mcalc::Mat<double>'"
def patches
# makes Scan Tailor work with Clang on OS X Mavericks
"https://gist.github.com/muellermartin/8569243/raw/b09215037b346787e0f501ae60966002fd79602e/scantailor-0.9.11.1-clang.patch"
Copy link
Member

Choose a reason for hiding this comment

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

Has this patch been submitted to Scan Tailor?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it has been submitted to the mailing list and reviewed by the current maintainer. The patch is included in the scantailor repository, according to his mail in the list.

See conversation: http://sourceforge.net/p/scantailor/mailman/message/31884956/

Copy link
Member

Choose a reason for hiding this comment

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

Perfect! Can you note that it's been accepted and link to the conversation, then?

@mistydemeo
Copy link
Member

Thank you! Pulled, with a couple tweaks.

ehershey pushed a commit to ehershey/homebrew that referenced this pull request Apr 4, 2014
Fixes Homebrew#11817.
Closes Homebrew#27919.

Signed-off-by: Misty De Meo <mistydemeo@gmail.com>
yeled pushed a commit to yeled/homebrew that referenced this pull request Mar 10, 2015
Fixes Homebrew#11817.
Closes Homebrew#27919.

Signed-off-by: Misty De Meo <mistydemeo@gmail.com>
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants