Skip to content

Cross version tests broken #548

Closed
Closed

Description

@mhdawson

We have a bunch of failures in the testing across verions. Believe this must have been the commit that caused it:

#534

Activity

  1. mhdawson commented on Sep 23, 2019

    @mhdawson
    MemberAuthor
  2. mhdawson commented on Sep 23, 2019

    @mhdawson
    MemberAuthor

    failures

     CXX(target) Release/obj.target/binding/basic_types/number.o
      CXX(target) Release/obj.target/binding/basic_types/value.o
      CXX(target) Release/obj.target/binding/bigint.o
      CXX(target) Release/obj.target/binding/date.o
      CXX(target) Release/obj.target/binding/binding.o
      CXX(target) Release/obj.target/binding/buffer.o
    In file included from /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi.h:2082:0,
                     from ../date.cc:2:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h: In member function ‘bool Napi::Value::IsDate() const’:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h:389:58: error: ‘napi_is_date’ was not declared in this scope
       napi_status status = napi_is_date(_env, _value, &result);
                                                              ^
    In file included from /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi.h:2082:0,
                     from ../date.cc:2:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h: In static member function ‘static Napi::Date Napi::Date::New(napi_env, double)’:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h:683:57: error: ‘napi_create_date’ was not declared in this scope
       napi_status status = napi_create_date(env, val, &value);
                                                             ^
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h: In member function ‘double Napi::Date::ValueOf() const’:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h:701:28: error: ‘napi_get_date_value’ was not declared in this scope
           _env, _value, &result);
                                ^
    binding.target.mk:159: recipe for target 'Release/obj.target/binding/date.o' failed
    make: *** [Release/obj.target/binding/date.o] Error 1
    make: *** Waiting for unfinished jobs....
    In file included from /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi.h:2082:0,
                     from ../bigint.cc:2:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h: In member function ‘bool Napi::Value::IsDate() const’:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h:389:58: error: ‘napi_is_date’ was not declared in this scope
       napi_status status = napi_is_date(_env, _value, &result);
                                                              ^
    In file included from /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi.h:2082:0,
                     from ../bigint.cc:2:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h: In static member function ‘static Napi::Date Napi::Date::New(napi_env, double)’:
    /data/iojs/build/workspace/node-test-node-addon-api-new/nodes/rhel72-s390x/node-addon-api/napi-inl.h:683:57: error: ‘napi_create_date’ was not declared in this scope
       napi_status status = napi_create_date(env, val, &value);
  3. mhdawson commented on Sep 23, 2019

    @mhdawson
    MemberAuthor

    I think I may have broken when I did a nightly last week. I used v8.x instead of v8.x-staging which would have removed some of the functions which have not yet landed as experimental.

    New nightly to try to resolve: https://ci-release.nodejs.org/job/iojs+release/4698/

  4. mhdawson commented on Sep 23, 2019

    @mhdawson
    MemberAuthor

    Ok, new 10.x nightly seems to have fixed 10.x

    Full cross version run here: https://ci.nodejs.org/view/x%20-%20Abi%20stable%20module%20API/job/node-test-node-addon-api-LTS%20versions/484/

  5. mhdawson commented on Sep 23, 2019

    @mhdawson
    MemberAuthor

    Ok all runs were green, so closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions