Skip to content

Comments

Build simplify#1155

Closed
rpavlik wants to merge 2 commits intosolvespace:masterfrom
rpavlik:build-simplify
Closed

Build simplify#1155
rpavlik wants to merge 2 commits intosolvespace:masterfrom
rpavlik:build-simplify

Conversation

@rpavlik
Copy link
Contributor

@rpavlik rpavlik commented Dec 22, 2021

Builds on #1154

Some cleanups of the cmake files so they're easier to understand and edit. Also fixes a bunch of CMake warnings.

@rpavlik rpavlik mentioned this pull request Dec 22, 2021
@phkahler
Copy link
Member

Merge conflicts?

@ruevs
Copy link
Member

ruevs commented Dec 22, 2021

Yes, because @phkahler already merged #1154 and you included those two commits in here as well.

By the way your Eigen branch also has them.

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 22, 2021

wait, how did the merge happen that I got conflicts from child commits? Did the commits get squashed or something that changed the ID?

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 22, 2021

ok well fixed that.

@ruevs ruevs requested a review from ppd December 23, 2021 06:30
@ruevs
Copy link
Member

ruevs commented Dec 23, 2021

wait, how did the merge happen that I got conflicts from child commits? Did the commits get squashed or something that changed the ID?

I don't know but the commit IDs "always" end up different on the master branch after merging. I thought it was normal?

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 23, 2021

Only if rebased or squashed. If it's a straight "merge" then the commit ID should exist in the history. Anyway, this is ready to review.

Copy link
Member

@phkahler phkahler left a comment

Choose a reason for hiding this comment

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

Build system stuff of such complexity I like to defer to those who have done the majority of the work on that ;-)

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 24, 2021

Who has done most of it? I think this is one of the few projects that I contribute to that I haven't been the main person behind the build system maintenance... Usually that's how I get hooked. (That's why MeshLab has CMake instead of qmake now and can use system installed dependencies, etc)

@ruevs
Copy link
Member

ruevs commented Dec 24, 2021

Who has done most of it?

Whitequark. All the way back to 2015 when she added it 5db5f1e until she left.

Since I don't know anything about CMake - beyond the little I learned to build SolveSpace I can't help you with this.

@phkahler
Copy link
Member

Who has done most of it? I think this is one of the few projects that I contribute to that I haven't been the main person behind the build system maintenance...

Well since whitequark departed I'd say it's been @ppd followed by @vespakoen. I figured people knew that since someone requested their review on this one? ;-)

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 25, 2021

Oh I just clicked the suggested people from GitHub 😁

@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 27, 2021

Rebased for cleanliness.

@phkahler
Copy link
Member

@rpavlik Going to close this since the commits were merged as part of the PR that built on this one. You can reopen if that's not quite right.

@phkahler phkahler closed this Dec 30, 2021
@rpavlik
Copy link
Contributor Author

rpavlik commented Dec 30, 2021

Cool thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants