2013/10/29 David Cole <[email protected]>: >> Stephen Kelly <steveire@...> writes: >> > Do you have any thoughts on the CPACK_PACKAGE_CONTACT problem? > > Namely that >> >> > the debian generator and the BAR generator have differing > > expectations on >> >> > what such a thing would contain. Should we have separate >> > CPACK_PACKAGE_CONTACT_NAME and CPACK_PACKAGE_CONTACT_EMAIL > > variables? >> >> >> Eric, do you have any thoughts on this issue?
Sorry Steve, But my bandwidth on the CMake/CPack front is currently so low that I can barely follow CMake ML discussion correctly. >> I'll soon try to revive this. See the discussion so far at: >> >> http://thread.gmane.org/gmane.comp.programming.tools.cmake.devel/7530 I remember but I have no time to help further right now and for 3 up to 4 weeks from now. > What about using the typical format that contains both name and email > address in CPACK_PACKAGE_CONTACT? Would "David Cole <[email protected]>" work > if used in that context for both of these generators...? Or would that make > one of them choke? This is OK with Debian. Note that concerning Debian, CPACK_PACKAGE_CONTACT is used as a deafult value for CPACK_DEBIAN_PACKAGE_MAINTAINER which is mandatory. If ever the chosen "common" format for "CPACK_PACKAGE_CONTACT" is not suitable as-is for filling CPACK_DEBIAN_PACKAGE_MAINTAINER we may be able to automatically extract the relevant bits from CPACK_PACKAGE_CONTACT in order to put appropriate value in CPACK_DEBIAN_PACKAGE_MAINTAINER. Currently we simply do: # Maintainer: (mandatory) if(NOT CPACK_DEBIAN_PACKAGE_MAINTAINER) if(NOT CPACK_PACKAGE_CONTACT) message(FATAL_ERROR "CPackDeb: Debian package requires a maintainer for a package, set CPACK_PACKAGE_CONTACT or CPACK_DEBIAN_PACKAGE_MAINTAINER") endif() set(CPACK_DEBIAN_PACKAGE_MAINTAINER ${CPACK_PACKAGE_CONTACT}) endif() > References: > > http://stackoverflow.com/questions/2631065/what-is-the-format-of-email-header-name-email > http://tools.ietf.org/html/rfc2822#section-3.4 "David Cole <[email protected]>" works fine as-is for .deb. The regex shown on the Stackoverflow article is scary. I guess CPack does not have to "validate" that. -- Erk L'élection n'est pas la démocratie -- http://www.le-message.org -- Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Follow this link to subscribe/unsubscribe: http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
