Skip to content

solvespace: remove #25569

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

solvespace: remove #25569

wants to merge 1 commit into from

Conversation

whitequark
Copy link

Motivation for this change

I'm the current maintainer of solvespace. The version in nixpkgs is a very outdated port that should have never been merged by then-maintainer in the state that it was in, and is completely unusable for real-world work. Since I do not use NixOS, I cannot provide an updated recipe. However, on multiple occasions people ran this broken port with the expectation that it is usable software, and at least I'd like to prevent that.

@mention-bot
Copy link

@whitequark, thanks for your PR! By analyzing the history of the files in this pull request, we identified @the-kenny, @globin and @bjornfor to be potential reviewers.

Copy link
Member

@thoughtpolice thoughtpolice left a comment

Choose a reason for hiding this comment

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

This can't be merged as it must also be removed from all-packages.nix, or it will break everything.

That said, I'm using solvespace myself now since recently, so I'm willing to take a whack at just fixing this.

@whitequark
Copy link
Author

cc @edef1c @ben0x539

@whitequark
Copy link
Author

If you want to package something stable you want the last commit from the 2.x branch (now b1d87bf284b32e875c8edba592113e691ea10bcd), that includes the glibc fix as well.

@ben0x539
Copy link
Contributor

ben0x539 commented May 6, 2017

Sounds like @edef1c might have some work towards packaging a more recent version already.

@edef1c
Copy link
Member

edef1c commented May 6, 2017

PR coming in a few minutes or so, just gonna test the version upgrade first and some minor changes to my build process.

@edef1c
Copy link
Member

edef1c commented May 6, 2017

#25570 (that took a little longer than expected)

@whitequark whitequark closed this May 6, 2017
@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants