Skip to content

Fix V8 function in Value.h#186

Open
Zosoled wants to merge 1098 commits into
CauldronDevelopmentLLC:masterfrom
Zosoled:fix/utf8length
Open

Fix V8 function in Value.h#186
Zosoled wants to merge 1098 commits into
CauldronDevelopmentLLC:masterfrom
Zosoled:fix/utf8length

Conversation

@Zosoled

@Zosoled Zosoled commented Nov 20, 2025

Copy link
Copy Markdown

jcoffland and others added 25 commits September 16, 2025 13:59
fah-client now utilizes the sd-device API. Requires systemd 240 or later.
…-device

Enable systemd support only with sd-device present
This reverts commit dcd7e5d.

sd-device usage in fah-client was dropped.
…d-device

Revert "Enable systemd support only with sd-device present"
The /usr/lib/rpm/brp-suse.d/brp-15-strip-debug build root policy script on openSUSE (from the brp-check-suse package)
does not honor the __strip definition.

To ensure debug symbols are preserved, set the environment variable NO_BRP_STRIP_DEBUG=true when building with
debug enabled.
Build root policy modifies the fahctl shebang. On RHEL 8, this results in:

+ /usr/lib/rpm/redhat/brp-mangle-shebangs
mangling shebang in /usr/bin/fahctl from /usr/bin/env python3 to #!/usr/libexec/platform-python

Then, rpmbuild adds an automatic requirement dependency on /usr/libexec/platform-python.

This package fails to install on recent Fedora versions because platform-python no longer exists:

Failed to resolve the transaction:
Problem: conflicting requests
  - nothing provides /usr/libexec/platform-python needed by fah-client-8.5.4-1.x86_64 from @commandline
You can try to add to command line:
  --setopt=optional_metadata_types=filelists to load additional filelists metadata
  --skip-broken to skip uninstallable packages

Let's keep the shebang untouched, which will result in a dependency on /usr/bin/env.

Define __brp_mangle_shebangs as %{nil} instead of undefining it, as recommended by:

https://bugzilla.redhat.com/show_bug.cgi?id=1597664#c5
@jcoffland

Copy link
Copy Markdown
Member

This is version dependent. Accepting this PR would break builds for other systems. It would have to be wrapped in an ifdef on the v8 version. At what version was this change made? E.g.: https://github.com/CauldronDevelopmentLLC/cbang/blob/master/src/cbang/js/v8/Context.cpp#L54

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

Successfully merging this pull request may close these issues.

6 participants