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

libexpr: add missing dependency on 'flake/call-flake.nix.gen.hh' #9462

Merged
merged 2 commits into from
Nov 27, 2023
Merged

libexpr: add missing dependency on 'flake/call-flake.nix.gen.hh' #9462

merged 2 commits into from
Nov 27, 2023

Conversation

trofi
Copy link
Contributor

@trofi trofi commented Nov 27, 2023

Without the change build for eval.o fails occasionally as:

$ make src/libexpr/eval.o
  GEN    Makefile.config
  GEN    src/libexpr/primops/derivation.nix.gen.hh
  GEN    src/libexpr/fetchurl.nix.gen.hh
  GEN    src/libexpr/parser-tab.cc
  GEN    src/libexpr/lexer-tab.cc
src/libexpr/lexer.l:314: warning, -s option given but default rule can be matched
  CXX    src/libexpr/eval.o
src/libexpr/eval.cc:519:18: fatal error: flake/call-flake.nix.gen.hh: No such file or directory
  519 |         #include "flake/call-flake.nix.gen.hh"
      |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make: *** [mk/patterns.mk:3: src/libexpr/eval.o] Error 1

Noticed in NixOS/nixpkgs#269439

Motivation

Context

Priorities

Add 👍 to pull requests you find important.

Without the change build for `eval.o` fails occasionally as:

    $ make src/libexpr/eval.o
      GEN    Makefile.config
      GEN    src/libexpr/primops/derivation.nix.gen.hh
      GEN    src/libexpr/fetchurl.nix.gen.hh
      GEN    src/libexpr/parser-tab.cc
      GEN    src/libexpr/lexer-tab.cc
    src/libexpr/lexer.l:314: warning, -s option given but default rule can be matched
      CXX    src/libexpr/eval.o
    src/libexpr/eval.cc:519:18: fatal error: flake/call-flake.nix.gen.hh: No such file or directory
      519 |         #include "flake/call-flake.nix.gen.hh"
          |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    compilation terminated.
    make: *** [mk/patterns.mk:3: src/libexpr/eval.o] Error 1

Noticed in NixOS/nixpkgs#269439
Dependency is now entirely through the eval.cc rule.
All gen.hh deps are now there.
@roberth
Copy link
Member

roberth commented Nov 27, 2023

The call-flake.nix.gen.hh reference was moved in #9177.

@roberth roberth merged commit f018048 into NixOS:master Nov 27, 2023
8 checks passed
Copy link

Successfully created backport PR for 2.19-maintenance:

@trofi trofi deleted the eval.o-dependency branch November 27, 2023 18:36
tebowy pushed a commit to tebowy/nix that referenced this pull request Jul 11, 2024
libexpr: add missing dependency on 'flake/call-flake.nix.gen.hh'
(cherry picked from commit f018048)
Change-Id: I27211872ad75034ae3f04e18b2987f9968073d00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.19-maintenance Automatically creates a PR against the branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants