Merge lp:~zorba-coders/zorba/bug-1034582-macports-flex into lp:zorba

Proposed by Chris Hillery
Status: Merged
Approved by: David Graf
Approved revision: 10982
Merged at revision: 10982
Proposed branch: lp:~zorba-coders/zorba/bug-1034582-macports-flex
Merge into: lp:zorba
Diff against target: 46 lines (+11/-5)
1 file modified
cmake_modules/CMakeUseFlex.cmake (+11/-5)
To merge this branch: bzr merge lp:~zorba-coders/zorba/bug-1034582-macports-flex
Reviewer Review Type Date Requested Status
David Graf (community) Approve
Chris Hillery Approve
Review via email: mp+119458@code.launchpad.net

Commit message

Try to find FlexLexer.h associated with the flex binary being used.

To post a comment you must log in.
Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
Revision history for this message
Paul J. Lucas (paul-lucas) wrote :

Works for me.

Revision history for this message
David Graf (davidagraf) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job bug-1034582-macports-flex-2012-08-14T08-26-05.466Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'cmake_modules/CMakeUseFlex.cmake'
--- cmake_modules/CMakeUseFlex.cmake 2012-08-10 02:16:18 +0000
+++ cmake_modules/CMakeUseFlex.cmake 2012-08-14 01:34:19 +0000
@@ -29,7 +29,6 @@
2929
30IF(NOT FLEX_EXECUTABLE)30IF(NOT FLEX_EXECUTABLE)
31 MESSAGE(STATUS "Looking for flex")31 MESSAGE(STATUS "Looking for flex")
32 FIND_PATH(FLEX_INCLUDE_DIR FlexLexer.h)
33 FIND_PROGRAM(FLEX_EXECUTABLE flex)32 FIND_PROGRAM(FLEX_EXECUTABLE flex)
34 IF(FLEX_EXECUTABLE)33 IF(FLEX_EXECUTABLE)
35 EXEC_PROGRAM(34 EXEC_PROGRAM(
@@ -40,15 +39,22 @@
40 # Extract major and minor versions39 # Extract major and minor versions
41 STRING (REGEX REPLACE "[^0-9]*([0-9]+)..*" "\\1" FLEX_MAJOR_VERSION_TMP ${FLEX_VERSION})40 STRING (REGEX REPLACE "[^0-9]*([0-9]+)..*" "\\1" FLEX_MAJOR_VERSION_TMP ${FLEX_VERSION})
42 STRING (REGEX REPLACE "[^0-9]*[0-9]+\\.([0-9]+).*" "\\1" FLEX_MINOR_VERSION_TMP ${FLEX_VERSION})41 STRING (REGEX REPLACE "[^0-9]*[0-9]+\\.([0-9]+).*" "\\1" FLEX_MINOR_VERSION_TMP ${FLEX_VERSION})
43 STRING (REGEX REPLACE "[^0-9]*[0-9]+\\.[0-9]+\\.([0-9]+[a-z]*).*" "\\1" FLEX_REVISION_VERSION_TMP ${FLEX_VERSION}) 42 STRING (REGEX REPLACE "[^0-9]*[0-9]+\\.[0-9]+\\.([0-9]+[a-z]*).*" "\\1" FLEX_REVISION_VERSION_TMP ${FLEX_VERSION})
44 MESSAGE(STATUS "Found flex -- ${FLEX_EXECUTABLE}, version: " ${FLEX_MAJOR_VERSION_TMP} "." ${FLEX_MINOR_VERSION_TMP} "." ${FLEX_REVISION_VERSION_TMP})43 MESSAGE(STATUS "Found flex -- ${FLEX_EXECUTABLE}, version: " ${FLEX_MAJOR_VERSION_TMP} "." ${FLEX_MINOR_VERSION_TMP} "." ${FLEX_REVISION_VERSION_TMP})
45 SET (FLEX_MAJOR_VERSION ${FLEX_MAJOR_VERSION_TMP} CACHE STRING "The flex major version" FORCE)44 SET (FLEX_MAJOR_VERSION ${FLEX_MAJOR_VERSION_TMP} CACHE STRING "The flex major version" FORCE)
46 SET (FLEX_MINOR_VERSION ${FLEX_MINOR_VERSION_TMP} CACHE STRING "The flex minor version" FORCE)45 SET (FLEX_MINOR_VERSION ${FLEX_MINOR_VERSION_TMP} CACHE STRING "The flex minor version" FORCE)
47 SET (FLEX_REVISION_VERSION ${FLEX_REVISION_VERSION_TMP} CACHE STRING "The flex revision version" FORCE)46 SET (FLEX_REVISION_VERSION ${FLEX_REVISION_VERSION_TMP} CACHE STRING "The flex revision version" FORCE)
47
48 # Find the FlexLexer.h header. Try our best to find the one associated
49 # with the flex binary we're actually using.
50 GET_FILENAME_COMPONENT(_flex_path "${FLEX_EXECUTABLE}" PATH)
51 GET_FILENAME_COMPONENT(_flex_hint "${_flex_path}/../include" ABSOLUTE)
52 FIND_PATH(FLEX_INCLUDE_DIR FlexLexer.h HINTS "${_flex_hint}")
53
48 ELSE (FLEX_EXECUTABLE)54 ELSE (FLEX_EXECUTABLE)
49 SET (FLEX_MAJOR_VERSION "0")55 SET (FLEX_MAJOR_VERSION "0")
50 SET (FLEX_MINOR_VERSION "0")56 SET (FLEX_MINOR_VERSION "0")
51 SET (FLEX_REVISION_VERSION "0")57 SET (FLEX_REVISION_VERSION "0")
52 ENDIF(FLEX_EXECUTABLE)58 ENDIF(FLEX_EXECUTABLE)
53 SET (FLEX_VERSION_FULL ${FLEX_MAJOR_VERSION}.${FLEX_MINOR_VERSION}.${FLEX_REVISION_VERSION} CACHE STRING "The Flex version" FORCE)59 SET (FLEX_VERSION_FULL ${FLEX_MAJOR_VERSION}.${FLEX_MINOR_VERSION}.${FLEX_REVISION_VERSION} CACHE STRING "The Flex version" FORCE)
54ENDIF(NOT FLEX_EXECUTABLE)60ENDIF(NOT FLEX_EXECUTABLE)
@@ -74,7 +80,7 @@
74 COMMAND "${CMAKE_COMMAND}"80 COMMAND "${CMAKE_COMMAND}"
75 ARGS -E copy_if_different81 ARGS -E copy_if_different
76 "${FLEX_INCLUDE_DIR}/FlexLexer.h"82 "${FLEX_INCLUDE_DIR}/FlexLexer.h"
77 "${FLEXLEXERH}")83 "${FLEXLEXERH}")
78 SET(OUTFILE "${CMAKE_CURRENT_BINARY_DIR}/${PATH}/${PREFIX}.cpp")84 SET(OUTFILE "${CMAKE_CURRENT_BINARY_DIR}/${PATH}/${PREFIX}.cpp")
79 ADD_CUSTOM_COMMAND(85 ADD_CUSTOM_COMMAND(
80 OUTPUT "${OUTFILE}"86 OUTPUT "${OUTFILE}"

Subscribers

People subscribed via source and target branches