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

try compiling for sm 115 #5146

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

try compiling for sm 115 #5146

wants to merge 2 commits into from

Conversation

big-r81
Copy link
Contributor

@big-r81 big-r81 commented Jul 22, 2024

Simple starting point...

Got the following compile error:

==> couch (compile)
Compiling /Users/big-r/Documents/Developer/CouchDB/couchdb/src/couch/priv/couch_js/115/main.cpp
/Users/big-r/Documents/Developer/CouchDB/couchdb/src/couch/priv/couch_js/115/main.cpp:50:5: error: excess elements in struct initializer
    JS_GlobalObjectTraceHook
    ^~~~~~~~~~~~~~~~~~~~~~~~
/Users/big-r/Documents/Developer/CouchDB/couchdb/src/couch/priv/couch_js/115/main.cpp:255:5: error: cannot initialize a member subobject of type 'JSCSPEvalChecker' (aka 'bool (*)(JSContext *, JS::RuntimeCode, Handle<JSString *>)') with an lvalue of type 'bool (JSContext *, JS::HandleString)' (aka 'bool (JSContext *, Handle<JSString *>)'): different number of parameters (3 vs 2)
    csp_allows,
    ^~~~~~~~~~
2 errors generated.
ERROR: compile failed while processing /Users/big-r/Documents/Developer/CouchDB/couchdb/src/couch: rebar_abort
make: *** [couch] Error 1

Like @nickva found out in #4305 csp_allows gets an extra flag. I looked at this migration guide, but found nothing which is affecting us (really? idk!).

@big-r81
Copy link
Contributor Author

big-r81 commented Jul 22, 2024

Fixing the compilation errors and now getting the following error when running http://127.0.0.1:15984/_utils/#/verifyinstall:

[error] 2024-07-22T19:27:50.849320Z node1@127.0.0.1 <0.897.0> -------- OS Process Error <0.906.0> :: {<<"TypeError">>,{[]}}

So almost the same as for sm 102.

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.

1 participant