Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qwt: add qt to tool_requirements #12831

Closed
wants to merge 4 commits into from

Conversation

ericLemanissier
Copy link
Contributor

@ericLemanissier ericLemanissier commented Sep 5, 2022

fixes #12830

Specify library name and version: qwt/*


  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the conan-center hook activated.

@conan-center-bot

This comment has been minimized.

@ericLemanissier
Copy link
Contributor Author

@uilianries @jgsogo what does this error mean ? is it an infrastructure issue ?

@conan-center-bot

This comment has been minimized.

@jgsogo
Copy link
Contributor

jgsogo commented Sep 5, 2022

The facts

neon is an option of libpng and it is removed for some configurations:

recipes/libpng/all/conanfile.py

    @property
    def _has_neon_support(self):
        return "arm" in self.settings.arch

    ...

    def config_options(self):
        ...
        if not self._has_neon_support:
            del self.options.neon

(From the error message) it looks like some other package is trying to get the value of that (removed) neon option

The why/wtf

The two configurations that are failing are arch=armv8 so the option neon should be available. For some reason _has_neon_support has been evaluated to False and the option has been removed 😮 I will investigate it

@jgsogo
Copy link
Contributor

jgsogo commented Sep 5, 2022

Oh! I think I know what's going on, and @SSE4 knows as well 😭 . This is a known bug that we realized in the context of CMake and openssl IIRC (as it is a very common build-require).

Here we have a cross-building scenario and the package libpng appears in both contexts now that qt is being added to the build one. One of those contexts is removing option neon from libpng while the other isn't, but Conan is not capable of dealing with this situation, it uses the same set of options for both contexts. The option pack that contains neon is applied for both and complains in the context where that option has been removed.

I'm not aware of any workaround, besides adding back that option to libpng even if it is not used for some configurations. Maybe @SSE4 has a better memory than me and he remembers any other alternative.

@conan-center-bot

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2022

Hooks produced the following warnings for commit 4bc8ef2
qwt/6.1.6
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library '.\bin\qwt.dll' links to system library 'uxtheme' but it is not in cpp_info.system_libs.

@conan-center-bot
Copy link
Collaborator

Failure in build 7 (4bc8ef27e5b6729bf105197f5dba74ac5c0d17e8):

  • qwt/6.1.6@:
    CI failed to create some packages (All logs)

    Logs for packageID 02943e72e9fa7b6899f45484366fa7b8bb07e735:
    [settings]
    arch=x86_64
    build_type=Debug
    compiler=Visual Studio
    compiler.runtime=MDd
    compiler.version=16
    os=Windows
    [options]
    qwt:shared=True
    
    [...]
    moc ..\source_subfolder\src\qwt_dial.h 
    moc ..\source_subfolder\src\qwt_counter.h 
    moc ..\source_subfolder\src\qwt_knob.h 
    moc ..\source_subfolder\src\qwt_slider.h 
    Generating Code...
    moc ..\source_subfolder\src\qwt_thermo.h 
    moc ..\source_subfolder\src\qwt_wheel.h 
    moc_qwt_dyngrid_layout.cpp
    moc_qwt_magnifier.cpp
    moc_qwt_picker.cpp
    moc_qwt_panner.cpp
    moc_qwt_text_label.cpp
    Generating Code...
    Generating Code...
    moc_qwt_abstract_legend.cpp
    moc_qwt_legend.cpp
    moc_qwt_legend_label.cpp
    moc_qwt_plot_renderer.cpp
    moc_qwt_plot.cpp
    moc_qwt_plot_canvas.cpp
    moc_qwt_plot_panner.cpp
    moc_qwt_plot_zoomer.cpp
    moc_qwt_plot_picker.cpp
    moc_qwt_sampling_thread.cpp
    moc_qwt_plot_magnifier.cpp
    moc_qwt_scale_widget.cpp
    moc_qwt_abstract_slider.cpp
    moc_qwt_abstract_scale.cpp
    moc_qwt_analog_clock.cpp
    moc_qwt_compass.cpp
    moc_qwt_counter.cpp
    moc_qwt_dial.cpp
    moc_qwt_slider.cpp
    moc_qwt_knob.cpp
    moc_qwt_plot_glcanvas.cpp
    moc_qwt_thermo.cpp
    moc_qwt_wheel.cpp
    linking ..\lib\qwtd.dll 
    LINK : fatal error LNK1104: cannot open file 'C:\J\w\prod\BuildSingleReference\.conan\data\freetype\2.12.1\_\_\package\c69bad48274e7fc45c887cddc32aa0459f0a81e0\lib\freetyped.lib'
    qwt/6.1.6: 
    
    jom 1.1.3 - empower your cores
    
    jom: C:\J\w\prod\BuildSingleReference@2\.conan\data\qwt\6.1.6\_\_\build\02943e72e9fa7b6899f45484366fa7b8bb07e735\src\Makefile [..\lib\qwtd.dll] Error 1104
    jom: C:\J\w\prod\BuildSingleReference@2\.conan\data\qwt\6.1.6\_\_\build\02943e72e9fa7b6899f45484366fa7b8bb07e735\Makefile [sub-src-make_first-ordered] Error 2
    qwt/6.1.6: ERROR: Package '02943e72e9fa7b6899f45484366fa7b8bb07e735' build failed
    qwt/6.1.6: WARN: Build folder C:\J\w\prod\BuildSingleReference@2\.conan\data\qwt\6.1.6\_\_\build\02943e72e9fa7b6899f45484366fa7b8bb07e735
    ERROR: qwt/6.1.6: Error in build() method, line 110
    	self.run(f"{vcvars} && jom")
    	ConanException: Error 2 while executing set "VSCMD_START_DIR=%CD%" && call "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC/Auxiliary/Build/vcvarsall.bat" amd64 && jom
    
  • qwt/6.2.0@:
    Didn't run or was cancelled before finishing


Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@ghost
Copy link

ghost commented Sep 6, 2022

I detected other pull requests that are modifying qwt/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@ericLemanissier
Copy link
Contributor Author

in favor of #12845

@ericLemanissier ericLemanissier deleted the patch-1 branch September 6, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[package] qwt/6.2.0: Compilation error with gcc9 profile
3 participants