Bug#1041633: cmake: FindPython.cmake returns /usr/local/lib/python3.11/dist-packages for Python_SITEARCH

2023-07-21 Thread Brad King
On Fri, Jul 21, 2023 at 2:37 PM Brad King wrote:
> The behavior change came from this CMake merge request:
> * https://gitlab.kitware.com/cmake/cmake/-/merge_requests/8538
> because the distutils variant is deprecated.

This is now under discussion in an upstream CMake issue:

* https://gitlab.kitware.com/cmake/cmake/-/issues/25113

Thanks,
-Brad



Bug#1041633: cmake: FindPython.cmake returns /usr/local/lib/python3.11/dist-packages for Python_SITEARCH

2023-07-21 Thread Brad King
On Fri, Jul 21, 2023 at 11:21 AM Sebastiaan Couwenberg wrote:
> On bookworm distutils is still used which returns:
...
> On sid sysconfig is used which results:

The behavior change came from this CMake merge request:

* https://gitlab.kitware.com/cmake/cmake/-/merge_requests/8538

because the distutils variant is deprecated.

> To get the right path for the Debian python3 interpreter,
> you need to add 'deb_system':

With distutils a generic `plat_specific=True` option was enough
to get each platform's preferred behavior.  Does every client
of Python's sysconfig need to have a big dispatch block that
looks up the host platform and memorizes platform-specific
arguments?  A similar concern was raised here:

* https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=998739#22

but never resolved.

As a side note, adding `DEB_PYTHON_INSTALL_LAYOUT=deb`
to the environment resolves this locally.

-Brad



Bug#959064: ignition-transport FTBFS in testing.

2020-04-30 Thread Brad King

On 4/29/2020 10:40 PM, peter green wrote:

The behavior of pkg-config has changed
This lets though a -isystem parameter with a space which was previously 
suppressed
And the space in said parameter breaks cmake.


I think cmake should handle -isystem¹ and as this is reproducible without
ignition-transport, I'm reassigning this to cmake. 


This certainly seems like the long term solution.


This has been reported to CMake upstream:

* PkgConfig chokes with `-isystem` with pkg-config 0.29.2
  https://gitlab.kitware.com/cmake/cmake/-/issues/20652

-Brad



Bug#915148: [Pkg-cmake-team] Bug#915148: cmake: regression in ros-ros-comm build

2018-12-07 Thread Brad King
On 12/7/18 7:44 AM, Jochen Sprickerhof wrote:
>> Both are valid ways to link to the pthread library, which is all
>> the `-pthread` flag does when used to drive linking.
> 
> I don't agree. Quoting from my mail:
> 
> "Define additional macros required"

Macros are for compiling.  It is not used during linking.
Either way, `-lfoo` can appear for other libraries and
ros-catkin needs the fix.

-Brad



Bug#915148: [Pkg-cmake-team] Bug#915148: cmake: regression in ros-ros-comm build

2018-12-07 Thread Brad King
On 12/6/18 4:16 PM, Jochen Sprickerhof wrote:
> after reading up on this, I think this needs fixing in cmake.
> 
> So neither adding -lpthread,
> nor adding /usr/lib/x86_64-linux-gnu/libpthread.so
> seems correct to me.

Both are valid ways to link to the pthread library, which is all
the `-pthread` flag does when used to drive linking.

> I will revert the workaround in ros-catkin if you agree.

The patch here:

  
http://launchpadlibrarian.net/399812910/ros-catkin_0.7.14-7_0.7.14-7ubuntu1.diff.gz

is not a workaround.  The input to that logic could contain -lfoo
generated by other means for other libraries.  This case just
happened to expose the existing bug in that logic.

Actually looking at the applied fix

  
https://sources.debian.org/src/ros-catkin/0.7.14-8/cmake/templates/pkgConfig.cmake.in/#L118

that logic may need additional work to handle plain `-pthread`
or other link flags.

> [FindThreads] promotes CMAKE_THREAD_PREFER_PTHREAD to get -pthread

It's THREADS_PREFER_PTHREAD_FLAG for that.

CMake's FindBoost and FindThreads could use some work to use
`-pthread` more completely (perhaps with a policy to use it
by default on appropriate compilers), and there is an issue
related to this here:

  https://gitlab.kitware.com/cmake/cmake/issues/18638

However, that does not mean the above change should be reverted.

-Brad



Bug#813875: insighttoolkit4: FTBFS on i386 (member reference base type 'void' is not a structure or union)

2016-02-09 Thread Brad King
On 02/06/2016 06:34 AM, Gert Wollny wrote:
> As you can see, this  is actually a bug in castxml
[snip]
> would you please consider filing a bug upstream

I've recorded the issue upstream here:

https://github.com/CastXML/CastXML/issues/47

Please follow that for updates.

-Brad



Bug#667417: fixing __float128?

2012-07-27 Thread Brad King
On 07/27/2012 02:17 AM, Mathieu Malaterre wrote:
 On Fri, Jul 27, 2012 at 4:13 AM, Steve M. Robbins st...@sumost.ca wrote:
 Hey ... is there a trick to fix #654718 similar to that for __int128?
 
 #undef _GLIBCXX_USE_FLOAT128

No, that is not the same problem.  The __int128 fix told the
*standard library* headers not to use the type by setting its
own preprocessor symbols correctly.  In this case the problem
is in the fftw headers.

The fftw3.h header contains a big #if test to decide whether
the __float128 type is available.  It *already* contains an
exclusion for the Intel compiler.  It also needs an exclusion
for gccxml just as it would for any other compiler that defines
__GNUC__ but is not really the GNU compiler version it reports.

This is what I suggested in the original thread on our list:

 http://thread.gmane.org/gmane.comp.compilers.gccxml/602

Other projects have accepted upstream modifications for the
same reason, such as Boost:

 
http://svn.boost.org/svn/boost/tags/release/Boost_1_50_0/boost/config/compiler/gcc_xml.hpp

-Brad


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#675181: gccxml cannot parse limits under GCC 4.7.1

2012-07-26 Thread Brad King
On 07/26/2012 05:31 AM, Mathieu Malaterre wrote:
 reopen 675181
 severity grave
 retitle 675181 gccxml cannot parse limits under GCC 4.7.1
 forwarded 675181 http://www.gccxml.org/Bug/view.php?id=13372
 thanks
 
 Not sure if this is ideal to reopen the bug instead of creating a new
 one. But I believe doko intention was to kill #667417 with this gccxml
 bug. So continuing the effort.
 
 I can reproduce the __int128 issue with current gccxml package in
 debian. It seems it also affect upstream (not checked if this is
 fixed), since upstream bug tracker reports it as open on 2012/07/02.
 
 Brad could you please confirm bug 13372 in gccxml stills affect current 
 gccxml ?

It did until I just fixed it.  Thanks for reminding me.

-Brad


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680825: [Debichem-devel] Bug#680825: gromacs: FTBFS: mv: cannot stat `/«PKGBUILDDIR»/debian/gromacs-mpich/usr/lib/*.so': No such file or directory

2012-07-11 Thread Brad King
On 07/11/2012 03:40 AM, Evgeni Golov wrote:
 Dear cmake maintainers, could you please have a look at the bug (and 
 build log) and guess why 2.8.9 stopped building the libraries in the 
 mdrun target? Is it a bad cmake file or a regression in cmake?

It is a regression in CMake AFAICT.  I bisected it down to here:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7ced0732

Try adding the flag

 -DCMAKE_INSTALL_DEFAULT_COMPONENT_NAME=Unspecified

to the CMake configuration step to work around the problem.
We'll try to get this fixed before the final 2.8.9 release.

-Brad



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680825: [Debichem-devel] Bug#680825: gromacs: FTBFS: mv: cannot stat `/«PKGBUILDDIR»/debian/gromacs-mpich/usr/lib/*.so': No such file or directory

2012-07-11 Thread Brad King
On 07/11/2012 02:29 PM, Brad King wrote:
 Try adding the flag
 
  -DCMAKE_INSTALL_DEFAULT_COMPONENT_NAME=Unspecified
 
 to the CMake configuration step to work around the problem.

Nevermind about this workaround.  I had tested it with a leftover
build of a good version during git bisect.

This hunk:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7ced0732#patch4

 @@ -35,7 +36,9 @@ cmInstallCommandArguments::cmInstallCommandArguments()
  ,NamelinkOnly  (Parser, NAMELINK_ONLY , ArgumentGroup)
  ,NamelinkSkip  (Parser, NAMELINK_SKIP , ArgumentGroup)
  ,GenericArguments(0)
 -{}
 +{
 +  this-Component.SetDefaultString(defaultComponent.c_str());
 +}

Seems to have reversed a previous fix:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=43cad3e4

of a problem similar to what we observe here.  The SetDefaultString
code never worked AFAICT.

-Brad



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680825: [Debichem-devel] Bug#680825: gromacs: FTBFS: mv: cannot stat `/«PKGBUILDDIR»/debian/gromacs-mpich/usr/lib/*.so': No such file or directory

2012-07-11 Thread Brad King
On 07/11/2012 02:55 PM, Brad King wrote:
 This hunk:
  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7ced0732#patch4
 Seems to have reversed a previous fix:
  http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=43cad3e4
 of a problem similar to what we observe here.

This should fix it:

 http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=8720aa04

Ideally we should add a test for this case but I have no time now.
We'll include the fix in the next 2.8.9 rc.

-Brad



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org