Why not review and merge the refactored beman-install-module.cmake?

We need this to start working and installing of Beman libraries as a CXX_MODULE

I plan on doing this this weekend.

1 Like

But you only started a series of merge request against my PR?

I open many issues in beman projects, most without any reactions.

I opened many PR and beman projects, most ended in weeks waiting to review, and if reviewed, they mostly ended in endless discussions until I closed them.

And in the beman discussion forum, also there is a response is not common since weeks?

That is why I am out of patience.

@ClausKlein I’m sorry to say it, but the quality of your PRs and issues are quite lacking. In many cases they are doing too many things at once and in others there is very little communication of intent. In the PR you mentioned above, for example, you didn’t replace the existing function like we had agreed upon in the modules sync up.

Keep in mind that folks here are volunteers and that you are not owed PR reviews or responses. I know this can be frustrating and I do appreciate your contributions, but we need to be respectful of other people’s time.

This has been merged.