Skip to content
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

Cmake config and project layout #26

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Mezozoysky
Copy link

@Mezozoysky Mezozoysky commented Nov 25, 2022

Added the ability to install signals as a library and to look for it using find_package(FTengSignals), etc..
United everything under one umbrella CMake-project.
Targets are FTengSignals::Signals, FTengSignals::TestApp, FTengSignals::Benchmarks.

@Mezozoysky
Copy link
Author

Looks like nobody home :-|

@dumblob
Copy link

dumblob commented Dec 7, 2022

Well, people are following this repo but @TheWisp does not seem very active (last activity on GitHub was in December 2021 when he/she forked https://github.com/TheWisp/coroutine_experiments from upstream).

So I would leave this PR open and maybe one day this repo will make it under some organization or it will be resurrected by @TheWisp 😉.

@Mezozoysky
Copy link
Author

Mezozoysky commented Dec 15, 2022

Hi, @dumblob!
I made sure the repository had been inactive for a while and merged this PR into my fork. Since license is MIT I plan to continue developing here https://github.com/Mezozoysky/FTengSignals.

@dumblob
Copy link

dumblob commented Dec 15, 2022

That's good news @Mezozoysky !

Could you turn on the "Issues" tab in your repository so that we can discuss some potential bugs, suggestions, improvements etc.?

I have commented in this original repo on several issues which I consider having the highest priority: https://github.com/TheWisp/signals/issues?q=is%3Aissue+involves%3Adumblob . I could then transfer these suggestions to your new repo to have some roadmap starting point if you like.

@Mezozoysky
Copy link
Author

I've turned on "Issues" and "Discussions", You welcome!

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.

2 participants