Incidencia #44422

macOS: --with-qt6-framework-bin doing nothing

Abrir Fecha: 2022-04-19 11:52 Última actualización: 2022-05-29 04:47

Informador:
Propietario:
Tipo:
Estado:
Cerrado
Componente:
Prioridad:
5 - Medium
Gravedad:
5 - Medium
Resolución:
Fixed
Fichero:
2

Details

As reported by jdlh in #44412, --with-qt5-framework-bin / --with-qt6-framework-bin configure options have no implementation at all - there's just the helptext for them.

Ticket History (3/11 Histories)

2022-04-19 11:52 Updated by: cazfi
  • New Ticket "macOS: --with-qt6-framework-bin doing nothing" created
2022-04-19 12:32 Updated by: jdlh
Comentario

I am able to build all the Freeciv branches on my Mac. If there are mac-specific things which need testing, you are welcome to assign them to me and I will eventually test it.

2022-04-19 13:37 Updated by: cazfi
Comentario

In addition to the implementation being empty, there's second point in jdlh's report that I almost missed. The value that the configure option is supposed to set is only used in the search for 'moc'. It's not used when looking for 'qtpaths'. It might have been an intentional design decision as 'qtpaths' is supposed to provide the *default* auto-detected paths that the configure option can override, but that design is broken in several ways (including the fact that 'qtpaths' is currently used also for things other than getting the bin directory)

CI macOS build scripts have overcome these issues by explicitly setting MOCCMD, and I guess things like CPPFLAGS and LIBS, so they don't really do any search for Qt at all. Should clean those script implementations once we get this low-level part to work correctly.

2022-04-26 11:26 Updated by: jdlh
Comentario

I am able to build all the Freeciv branches on my Mac. If there are mac-specific things which need testing, you are welcome to assign them to me and I will eventually test it.

2022-04-27 21:30 Updated by: cazfi
Comentario

I hope to get patch for this submitted later tonight, after doing some runs of CI mac build.

The interaction between --with-qt?-framework from #44412 and the --with-qt?-framework-bin might be a bit unclear.
With the implementation I currently have --with-qt-framework does not affect search of binaries at all - you need to use --with-qt-framework-bin for that.
If you only use --with-qt-framework-bin, that sets also the overall framework location with qtpaths.
If you use both options, the overall framework location is one explicitly given, and the ...-bin only affects search for the 'moc' -binary (qtpaths is not needed)

2022-04-27 23:18 Updated by: cazfi
  • Propietario Update from (Ninguno) to cazfi
  • Resolución Update from Ninguno to Accepted
Comentario

Patches attached. They are not breaking our existing CI builds, but I have had no setup to test if they improve the situation either. Can jdlh test if these fix the issues for him?

I'm not absolutely sure, but it's very likely that this depends on patch on #44412 (at least textually). Probably better to test with that one (which likely goes in first, anyway) applied first.

2022-05-12 04:31 Updated by: cazfi
Comentario

Reply To cazfi

Can jdlh test if these fix the issues for him?

Ping.

We may need to make the 3.0.2 release sooner than what we've planned so far.

2022-05-27 12:17 Updated by: cazfi
Comentario

I probably push this in soon, even if it doesn't get tested before. What didn't do anything before, cannot go more broken, at least.

2022-05-29 04:47 Updated by: cazfi
  • Estado Update from Open to Cerrado
  • Resolución Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Entrar