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

Revert "cmake standalone" #435

Closed
wants to merge 6 commits into from
Closed

Conversation

priseborough
Copy link
Collaborator

@priseborough priseborough commented May 4, 2018

Reverts #432

Requested because #432 breaks builds for projects forked prior to PX4/PX4-Autopilot@2e92484 that wish to use latest ecl and has not been peer reviewed.

@priseborough priseborough requested a review from dagar May 4, 2018 04:37
@dagar
Copy link
Member

dagar commented May 4, 2018

PX4/Firmware master ECL was updated ~ 16 hours ago to the latest PX4/ecl master commit before the cmake changes, so projects forked prior to PX4/PX4-Autopilot@2e92484 aren't stuck.

image

Is there something broken that can't be fixed with updating both Firmware and ecl?

@priseborough
Copy link
Collaborator Author

The problem i have with the project I am working on at the moment is that the changes made by this PR break the build and there are a number of conflicts with PX4/PX4-Autopilot@2e92484 that I am now working through.

@dagar
Copy link
Member

dagar commented May 5, 2018

Closing the PR, but keeping the branch for reference.

@dagar dagar closed this May 5, 2018
@priseborough priseborough deleted the revert-432-pr-cmake_standalone branch May 20, 2018 09:09
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.

2 participants