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

fix installation of main module headers #797

Merged
merged 1 commit into from
May 22, 2013

Conversation

vhdirk
Copy link
Contributor

@vhdirk vhdirk commented Apr 10, 2013

see: http://code.opencv.org/issues/2897

The order of the 2 conditions is really important, as every MATCHES operator sets ${CMAKE_MATCH_1}, which is needed inside the condition block.
Ideally, this should be done using a single regex with a negative forward lookahead, but CMake doesn't seem to support this...

see: http://code.opencv.org/issues/2897

The order of the 2 conditions is really important, as every MATCHES
operator sets ${CMAKE_MATCH_1}, which is needed inside the condition
block. Ideally, this should be done using a single regex with a
negative forward lookahead, but CMake doesn't seem to support this
@ghost ghost assigned taka-no-me Apr 11, 2013
@mdim
Copy link
Contributor

mdim commented Apr 11, 2013

@taka-no-me Andrey, can you please review the request?

@ghost ghost assigned vpisarev May 6, 2013
@AnnaPetrovicheva
Copy link
Contributor

Vadim, could you please review this pull request?

@Foaly
Copy link

Foaly commented May 6, 2013

I just experienced the same problem reported on the issue tracker and the proposed fix solved the problem perfectly! Im on a windows 8 machine with MinGW gcc 4.7.2

@alekcac
Copy link
Contributor

alekcac commented May 17, 2013

Vadim, any updates?

@apavlenko
Copy link
Contributor

👍

apavlenko pushed a commit that referenced this pull request May 22, 2013
@opencv-pushbot opencv-pushbot merged commit caa9ac4 into opencv:master May 22, 2013
@vhdirk vhdirk deleted the installheaders branch May 22, 2013 10:40
savuor pushed a commit to nickyu-zhu/opencv that referenced this pull request Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants