Skip to content

Build break in src/tracing/agent.h #18361

Closed

Description

@kfarnung
  • Version: master
  • Platform: windows, linux
  • Subsystem: tracing

It looks like #17489 and #18196 collided in master to break the build. The CurrentTimestampMicroseconds method was removed from the V8 type just as we took a dependency on it.

E:\GitHub\node\src\tracing/agent.h(15): error C3668: 'node::tracing::TracingController::CurrentTimestampMicroseconds': method with override specifier 'override' did not override any base class methods (compiling source file src\tracing\agent.cc) [E:\GitHub\node\node_lib.vcxproj]

/cc @ofrobots @targos @MylesBorins

Activity

  1. kfarnung commented on Jan 25, 2018

    @kfarnung
    ContributorAuthor

    Looks like #18360 is open to resolve this.

  2. apapirovski commented on Jan 25, 2018

    @apapirovski
    Contributor

    Fixed now that #18360 landed.

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