Skip to content

Fix standalone service compilation#77

Closed
mauriciovasquezbernal wants to merge 1 commit intopolycube-network:masterfrom
mauriciovasquezbernal:pr/fix_standalone_service_compilation
Closed

Fix standalone service compilation#77
mauriciovasquezbernal wants to merge 1 commit intopolycube-network:masterfrom
mauriciovasquezbernal:pr/fix_standalone_service_compilation

Conversation

@mauriciovasquezbernal
Copy link
Contributor

Fixes the compilation of standalone services.

Wrong spdlog library includes were causing a compilation failure
for standalone services.

Fixes: 9d00993 ("create base cube")

Signed-off-by: Mauricio Vasquez B <mauriciovasquezbernal@gmail.com>
@mauriciovasquezbernal mauriciovasquezbernal force-pushed the pr/fix_standalone_service_compilation branch from cc10df2 to dd6b7c5 Compare March 4, 2019 13:11
Copy link
Contributor

@mbertrone mbertrone left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

@acloudiator acloudiator requested a review from goldenrye March 6, 2019 00:22
@acloudiator acloudiator removed the request for review from goldenrye March 7, 2019 22:32
@acloudiator acloudiator added this to the PCN-Framework milestone Mar 7, 2019
@acloudiator
Copy link
Contributor

acloudiator commented Mar 13, 2019

@mauriciovasquezbernal I see the label do not merge by you. Do you still want to hold this PR for merge?

@mauriciovasquezbernal
Copy link
Contributor Author

I think this is still incomplete, so hold this for now.

@mauriciovasquezbernal
Copy link
Contributor Author

This solution is just a workaround, so I prefer to close it until we implement a proper solution for this problem: #91.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants