Skip to content

CMake updates#505

Merged
rgov merged 4 commits into
stp:masterfrom
MartinNowack:build_fixes
Oct 23, 2025
Merged

CMake updates#505
rgov merged 4 commits into
stp:masterfrom
MartinNowack:build_fixes

Conversation

@MartinNowack

Copy link
Copy Markdown
Contributor

A couple of build fixes/enhancements

@rgov

rgov commented Aug 29, 2025

Copy link
Copy Markdown
Member

Sorry for not seeing this earlier.

@nindanaoto What does linking to tinfo accomplish?

@nindanaoto

Copy link
Copy Markdown
Contributor

Well, I found that tinfo seems to be not necessary. As far as I remember, this was required by ABC, but removing that still goes well. I'm not sure about why I added this. May be this was my fault, and removing that is better.

@ccadar

ccadar commented Oct 23, 2025

Copy link
Copy Markdown
Contributor

@rgov May I ping you about these changes?

We are having problems using the current STP in KLEE, and these changes would help us with that.

Thank you!

@rgov rgov merged commit 14fd2f8 into stp:master Oct 23, 2025
2 checks passed
@rgov

rgov commented Oct 23, 2025

Copy link
Copy Markdown
Member

@ccadar Nice to hear from you. Sorry for the delay. I've tried to adapt the "Don't link with tinfo" commit according to the latest CMake configuration. CI builds passed, so it seems to work... let me know if you have any trouble.

@ccadar

ccadar commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

@rgov many thanks for the quick response!

I think there is another issue here with installing libabc-pic.so, and I will propose a PR shortly. Thanks again!

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.

4 participants