[MacPorts] #52927: cpplocate submission

MacPorts noreply at macports.org
Wed Dec 21 19:59:54 CET 2016


#52927: cpplocate submission
------------------------------------+--------------------------------
  Reporter:  ken-cunningham-webuse  |      Owner:  macports-tickets@…
      Type:  submission             |     Status:  new
  Priority:  Normal                 |  Milestone:
 Component:  ports                  |    Version:  2.3.5
Resolution:                         |   Keywords:
      Port:  cpplocate              |
------------------------------------+--------------------------------

Comment (by michaelld):

 There's a little more work to be done. I think all of this should be
 pushed back upstream, if nothing else than for their information. I'm
 going to attach a tarball of the current whole proposed directory
 structure, with my updates listed below in place. So ... IMHO, you need to
 address (3) for full port functionality.

 1) I'm not sure what SYSTEM_DIR_INSTALL is supposed to do internally, but
 checking for just /usr and /usr/local is pretty limiting. There has got to
 be a better way to do whatever they think they are doing with this
 variable that is more generic. Your patch is good enough for MacPorts'
 needs.

 2) When building documentation with doxygen, also include
 path:bin/dot:graphviz for dot (graph) files; I added this in already.

 3) cmake files are usually placed into ${prefix}/share/cmake . No so for
 this port by default:
 {{{
 $ port contents cpplocate | grep cmake
   /opt/local/share/cpplocate/cmake/ModuleInformation.cmake
   /opt/local/share/cpplocate/cmake/cpplocate/cpplocate-export-
 release.cmake
   /opt/local/share/cpplocate/cmake/cpplocate/cpplocate-export.cmake
   /opt/local/share/cpplocate/cpplocate-config.cmake
 }}}
 I haven't looked into how to change this install directory, but these devs
 seem to make such changes pretty simple. I'd much prefer to see the cmake
 files in their correct directory rather than a project-specific one; just
 easier to find. Ideally this would be set via configure.args as a way to
 override the default provided. See, e.g., gr-ais and the variable
 CMAKE_MODULES_DIR in the patch and top-level CMakeLists.txt for one way to
 do this.

--
Ticket URL: <https://trac.macports.org/ticket/52927#comment:6>
MacPorts <https://www.macports.org/>
Ports system for macOS


More information about the macports-tickets mailing list