[Debian-pan-maintainers] McCode / Debian

Emmanuel FARHI emmanuel.farhi at synchrotron-soleil.fr
Wed Jul 17 12:52:41 BST 2024


Hi guys,

The McCode packaging is now nearly finalized for Debian. However there 
are still a few missing files, and I think I have understood the reason 
for this behaviour. I have the feeling that the build directory in the 
Debian chroot (sbuild) can cause some confusion in this macro.

The /cmake/Modules/MCUtil.cmake/ file defines the /file_globsrc/ macro 
(line 308) which purpose is to scan a list of files and return it into a 
variable. But, the temporary chroot built for the preparation of the 
packages has the form:

    /<<PKGBUILDDIR>>/debian/package/...

and perhaps the MATCHES regexp are confused with the << and >> symbols 
therein (via the "${f}").

The only place where /file_globsrc /remains are in the 
/mcxtrace/CMakeLists.txt/ and /mcstas/CMakeLists.txt/. This may be the 
reason for some missing libraries which indeed are shipped via the 
'/mcxtrace/' and '/mcstas/' packages, namely read_table-lib and other 
libs in e.g. /mcxtrace/lib/share/ (which is as well a link to 
/common/lib/share//).

Some of the /file_globsrc/ calls have already been replaced by simpler:

  * file(GLOB VAR DIR/*)

statements. I'd like to replace these last occurrences in file 
"/cmake/Modules/InstallMCCODE.cmake/", namely:

  * file_globsrc( general_codefiles "${WORK}/lib/share/*.h"
    "${WORK}/lib/share/*.c" )
  * file_globsrc( flavor_codefiles "${FLAVOR_LIB}/share/*.h"
    "${FLAVOR_LIB}/share/*.c" "${FLAVOR_LIB}/share/*.cl" )

by simpler series of:

  * file(GLOB VAR DIR/*)

The lib/share directory only contains .h, .c and .cl files. So the 
inclusion via a file(GLOB VAR DIR/*) should work straight away.

What do you think ?

Emmanuel.

-- 
signature
/ ___|__/\_| |   | ____|_ _| |      FARHI Emmanuel
\___ \\ | |   |  _|  | || |      Div Exp/Data Reduction and Analysis Team
  ___) /_  _| |___| |___ | || |___   Tel : +33 (1) 69 35 96 04 
<callto:+33%20%281%29%2069%2035%2096%2004>
|____/  \/ |_____|_____|___|_____|  Saint-Aubin BP 48 - 91192 GIF/YVETTE 
CEDEX
SYNCHROTRON http://www.synchrotron-soleil.fr
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-pan-maintainers/attachments/20240717/562647bf/attachment.htm>


More information about the Debian-pan-maintainers mailing list