Arch Linux User Repository

This package is required for the R-package V8 (https://github.com/jeroen/V8). It is frequently updated, though unless it is broken, users must not build all updates. Since V8 contains a large codebase and compilation takes a while, I recommend to build it in a directory and not with an AUR-helper.

[Edit:] For binary builds please see: https://garbuszus.de/archpkgs

Yes, I just tested and that will work for me. The odd thing, though, is the file,v8-primitive.h, has 13 other occurrences of V8_ENABLE_CHECKS where they use #ifdef instead of #if. Still, probably easiest to change the header in my package since it works either way.

Could the same be done for the include diff we added earlier? #define V8_ENABLE_SANDBOX 1?

I think you are right. One of the headers in the package I am maintaining has a #define V8_ENABLE_CHECKS. If I change it to #define V8_ENABLE_CHECKS 1 it fixes the error. Thank you.

Thanks for the report. I am just wondering if V8 expects -DV8_ENABLE_CHECKS=0 and similar for preprocessor flags. They use both if and ifdef for V8_ENABLE_CHECKS, but there’s nothing stopping you from passing -DV8_ENABLE_CHECKS=0 which would be true for #ifdef V8_ENABLE_CHECKS. After all this might be something that the embedder could/should solve and patching wasn’t the right thing to do in the first place?

Hi. I have another error: #if with no expression, this time in the file /usr/include/v8-primitive.h, line 610 & 616. Again, changing the #if to #ifdef fixes the issue. I don't know exactly when this was introduced. The last version I built was 12.7.171.

That should be possible, but depot-tools is a collection of software that is used almost exclusively for Chrome development and probably has no use outside of Google. The AUR packages for depot-tools were often barely maintained (someone put them in the AUR and later probably lost interest because ... nobody really has any use for them). Since the PKGBUILD has been like this for five years, is there a particular reason why you want to see this changed? You need internet access either way to retrieve the package sources, so I don't really see the advantage. You're also welcome to use the binaries for v8-r from Github or my small Arch repository if you don't want to pollute your PATH variable. This also saves about 15 minutes of the computer building the package with all available resources.

Hi, is it possible to use the depot-tools-git package instead of cloning the depot_tools repository?

Sure, I'll add it to the next build. If you want binaries, but not built by me, you can also have a look at github where I mirror the package and build it via github actions e.g. https://github.com/JanMarvin/v8-R/releases/tag/12.2.176-1

Can you add procps-ng to the makedepends array? This is needed to build in a clean chroot. I originally didn't think it should be needed since it is a part of base, but apparently that's not the case. See my forum post here

Thanks for reaching out. The change was introduced recently. I might patch it for tomorrows release (V8 upstream is usually not that responsive to non Google issues)