Re: Problem with cmake 3.23.0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 11, 2022 at 11:15 PM Michel Alexandre Salim
<salimma@xxxxxxxxxxxxxxxxx> wrote:
>
> On Fri, Mar 04, 2022 at 09:26:04AM -0500, Steven A. Falco wrote:
> > There is a new FTBFS for KiCad [1].  I filed an issue with KiCad [2] and got a comment from the project leader:
> >
> >     This looks like cmake issue to me. For some reason cmake is creating an incorrect build folder:
> >
> This is probably related: in cmake 3.22 (Fedora 36) and below, this
> works:
>
> ```
> mkdir some-dir
> cd some-dir
> %cmake ..
> cd -
>
> mkdir other-dir
> cd other-dir
> %cmake ..
> cd -
> ```
>
> Which is useful if you need to invoke `%cmake` several times, e.g. to
> build against different Lua runtimes (e.g. lua-luv) or to do both static
> and shared library builds (which we used to do for the folly stack).
>
> Also: if the CMakeLists.txt is not in the root of your project tarball,
> previously you can just do `%cmake actual-src-dir`.
>
> In F37's cmake 3.23, it seems that `cmake -S . ..` has the `-S .` trump
> the `..` when it comes to finding where CMakeLists.txt is, while
> previously `..` wins.
>
> I was working around it by simply `cd actual-src-dir` before `%cmake`,
> but it turns out simply passing an additional `-S` works, e.g.:
>
> ```
> mkdir some-dir
> cd some-dir
> %cmake -S ..
> cd -
> ```
>
> e.g.
> https://src.fedoraproject.org/rpms/lua-luv/pull-request/3

This makes me wonder.

A lot of the things that are now reported as "broken" look like they
should already have been broken when we switched to out-of-tree builds
for CMake by default.
And a lot of the snippets look like things that I helped fix when that
change landed.

So ... did anybody investigate what actually changed here? Was it a
behaviour change in cmake CLI argument handling, or have the actual
%cmake / %cmake_build etc. macros changed in a way that makes them no
longer backwards compatible?

> If you mean the Fedora 33 change page, it only describes the change
> from Fedora 33+ perspective.  It does not take compatibility with EPEL
> buildroots into account at all.

Also related, as far as I know, Neal put great effort into making the
macros work almost exactly the same on both Fedora and EPEL.
So I think that statement is incorrect.

Fabio
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux