Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Fixed nebula-common dependency #4

Merged
merged 2 commits into from
Mar 9, 2020
Merged

Fixed nebula-common dependency #4

merged 2 commits into from
Mar 9, 2020

Conversation

sherman-the-tank
Copy link
Member

Now the build will work even without building the nebula-comon in advance

Now the build will work even without building the nebula-comon in advance
@sherman-the-tank sherman-the-tank changed the title Fixed bebula-common dependency Fixed nebula-common dependency Feb 29, 2020
dutor
dutor previously approved these changes Mar 2, 2020
Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

SET(NEBULA_COMMON_PACKAGE "nebula-common")

# Configure the dependent projects
include(AddDependentProject)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't see the AddDependentProject.cmake file.

@dutor dutor merged commit b7d492b into vesoft-inc:master Mar 9, 2020
sherman-the-tank added a commit that referenced this pull request Mar 31, 2020
sherman-the-tank added a commit that referenced this pull request Mar 31, 2020
* Fixed bebula-common dependency

Now the build will work even without building the nebula-comon in advance

* Added missing file
@bright-starry-sky bright-starry-sky mentioned this pull request May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants