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

qt-5: add buildPackages.stdenv.cc to nativeBuildInputs #220310

Closed
wants to merge 5 commits into from
Closed

qt-5: add buildPackages.stdenv.cc to nativeBuildInputs #220310

wants to merge 5 commits into from

Commits on Mar 10, 2023

  1. qt-5: add buildPackages.stdenv.cc to nativeBuildInputs

    Qmake expects to be able to find the host's C compiler in the $PATH
    when cross compiling.  Let's add it.
    
    Co-authored-by: superherointj <5861043+superherointj@users.noreply.github.com>
    Adam Joseph and superherointj committed Mar 10, 2023
    Configuration menu
    Copy the full SHA
    cfa4de5 View commit details
    Browse the repository at this point in the history

Commits on Mar 21, 2023

  1. Update pkgs/development/libraries/qt-5/5.15/default.nix

    Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
    Adam Joseph and SuperSandro2000 authored Mar 21, 2023
    Configuration menu
    Copy the full SHA
    35e1d46 View commit details
    Browse the repository at this point in the history
  2. Update pkgs/development/libraries/qt-5/5.15/default.nix

    Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
    Adam Joseph and SuperSandro2000 authored Mar 21, 2023
    Configuration menu
    Copy the full SHA
    65c3874 View commit details
    Browse the repository at this point in the history
  3. Update pkgs/development/libraries/qt-5/qtModule.nix

    Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
    Adam Joseph and SuperSandro2000 authored Mar 21, 2023
    Configuration menu
    Copy the full SHA
    393ab31 View commit details
    Browse the repository at this point in the history
  4. qt5: move buildPackages.stdenv.cc to pkgsBuildBuild

    As recommended here:
    #220310 (comment)
    
    The non-idiomatic `if..then..else null` is to prevent a mass-rebuild
    sending this to staging.  I can submit a separate PR to staging to
    change it to the `lib.optional` idiom if this bothers people.
    Adam Joseph committed Mar 21, 2023
    Configuration menu
    Copy the full SHA
    ceb4d99 View commit details
    Browse the repository at this point in the history