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

Port absl config to gyp #211

Closed
targos opened this issue Sep 21, 2021 · 12 comments
Closed

Port absl config to gyp #211

targos opened this issue Sep 21, 2021 · 12 comments

Comments

@targos
Copy link
Member

targos commented Sep 21, 2021

https://github.com/nodejs/node-v8/actions/runs/1256415634

In file included from ../deps/v8/src/base/platform/mutex.h:10,
                 from ../deps/v8/src/base/platform/condition-variable.h:10,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.h:13,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.cc:5:
../deps/v8/src/base/optional.h:8:10: fatal error: absl/types/optional.h: No such file or directory
    8 | #include "absl/types/optional.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
@targos targos changed the title Port absl config Port absl config to gyp Sep 21, 2021
@targos
Copy link
Member Author

targos commented Sep 21, 2021

This is an external dep to V8 that we'll have to pull.
Main BUILD.gn: https://chromium.googlesource.com/chromium/src/third_party/abseil-cpp/+/refs/heads/main/BUILD.gn

I probably won't have time to work on this in the near future.

@targos
Copy link
Member Author

targos commented Sep 21, 2021

I disabled the canary update GitHub action. It won't work until this is done.

@richardlau
Copy link
Member

Is this likely to block V8 9.5 for Node.js 17?

@targos
Copy link
Member Author

targos commented Sep 22, 2021

No, this is blocking V8 9.6

@richardlau
Copy link
Member

I've opened node-core-utils PR to add abseil to the fetched V8 dependencies on V8 updates: nodejs/node-core-utils#565

@richardlau
Copy link
Member

https://github.com/nodejs/node-v8/actions/runs/1256415634

In file included from ../deps/v8/src/base/platform/mutex.h:10,
                 from ../deps/v8/src/base/platform/condition-variable.h:10,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.h:13,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.cc:5:
../deps/v8/src/base/optional.h:8:10: fatal error: absl/types/optional.h: No such file or directory
    8 | #include "absl/types/optional.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

FWIW this particular use was reverted in v8/v8@d67f060

@targos
Copy link
Member Author

targos commented Sep 25, 2021

https://github.com/nodejs/node-v8/actions/runs/1256415634

In file included from ../deps/v8/src/base/platform/mutex.h:10,
                 from ../deps/v8/src/base/platform/condition-variable.h:10,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.h:13,
                 from ../deps/v8/src/libplatform/default-foreground-task-runner.cc:5:
../deps/v8/src/base/optional.h:8:10: fatal error: absl/types/optional.h: No such file or directory
    8 | #include "absl/types/optional.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

FWIW this particular use was reverted in v8/v8@d67f060

Right. It re-enabled the job as it seems V8 doesn't use absl anywhere after this revert.

@targos
Copy link
Member Author

targos commented Jul 18, 2022

Closing as this didn't show up again after almost a year.

@targos targos closed this as completed Jul 18, 2022
@targos
Copy link
Member Author

targos commented Nov 13, 2023

It's back:

In file included from ../../deps/v8/src/objects/source-text-module.h:11:
../../deps/v8/src/zone/zone-containers.h:20:10: fatal error: 'absl/container/flat_hash_map.h' file not found
#include "absl/container/flat_hash_map.h"
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
[1161/3998] CC obj/tools/icu/gen/icudata.icudt73_dat.o
ninja: build stopped: subcommand failed.
make: *** [node] Error 1

@targos targos reopened this Nov 13, 2023
@targos
Copy link
Member Author

targos commented Nov 13, 2023

v8/v8@fa7521e

targos added a commit to targos/node-core-utils that referenced this issue Nov 13, 2023
targos added a commit to nodejs/node-core-utils that referenced this issue Nov 15, 2023
@targos
Copy link
Member Author

targos commented Nov 15, 2023

I disabled https://github.com/nodejs/node-v8/actions/workflows/update-canary.yml. No point running a daily workflow that's guaranteed to fail until we fix this.

targos added a commit to nodejs/node that referenced this issue Nov 15, 2023
@targos
Copy link
Member Author

targos commented Nov 15, 2023

I went very manual and added a static library build for v8_abseil with the dependency tree of flat_hash_map and flat_hash_set in 1b0759c. It's probably overkill and might break in the future if V8 starts depending on other headers, but it works 🤷🏻.

@targos targos closed this as completed Nov 15, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 16, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 17, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 18, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 19, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 20, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 21, 2023
nodejs-github-bot pushed a commit that referenced this issue Nov 22, 2023
targos added a commit to nodejs/node that referenced this issue Nov 23, 2023
nodejs-github-bot pushed a commit that referenced this issue Mar 17, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 18, 2024
targos added a commit to nodejs/node that referenced this issue Mar 19, 2024
targos added a commit to targos/node that referenced this issue Mar 19, 2024
targos added a commit to targos/node that referenced this issue Mar 19, 2024
targos added a commit to targos/node that referenced this issue Mar 19, 2024
targos added a commit to targos/node that referenced this issue Mar 19, 2024
targos added a commit to nodejs/node that referenced this issue Mar 19, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 19, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 20, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 21, 2024
targos added a commit to targos/node that referenced this issue Mar 21, 2024
targos added a commit to targos/node that referenced this issue Mar 21, 2024
targos added a commit to targos/node that referenced this issue Mar 21, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 22, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 23, 2024
targos added a commit to targos/node that referenced this issue Mar 23, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 24, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 25, 2024
targos added a commit to nodejs/node that referenced this issue Mar 25, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 25, 2024
targos added a commit to nodejs/node that referenced this issue Mar 26, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 26, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 27, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 28, 2024
targos added a commit to nodejs/node that referenced this issue Mar 29, 2024
targos added a commit to targos/node that referenced this issue Mar 30, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 30, 2024
nodejs-github-bot pushed a commit that referenced this issue Mar 31, 2024
targos added a commit to nodejs/node that referenced this issue Mar 31, 2024
Refs: v8/v8@fa7521e
Closes: nodejs/node-v8#211
PR-URL: #51362
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants