[Default] On Fri, 16 Apr 2021 08:57:06 -0500, Matt Pavlovich <[email protected]> wrote:
| I anticipate an updated CMS library would be very useful to many | ActiveMQ users for years to come. The utility to users is beyond question. The issue is the interest of the developers/gatekeepers, who can forestall or prevent improvements, by simply doing nothing. If they have genuinely lost interest... There are already 9 outstanding PRs, going back several years. Getting rid of std::auto_ptr is only staving off an immediate problem (which really should have been addressed years ago, considering how simple the fix is.) But there are other serious issues with the code base that also need to be addressed, sooner rather than later. If something as not much more than cosmetic as replacing std::auto_ptr doesn't find favor with the gatekeepers, I don't fancy the chances for any serious changes. | 1. Create a clone of this git repo activemq-cpp <https://github.com/apache/activemq-cpp> This clone is in github itself, right? | 2. Create a branch (using the JIRA name is good ie AMQCPP-664) And this is in a local copy of my github clone, right? | 3. Make the code changes | 4. Push the branch to your cloned repo. | 5. GitHub will prompt you to create a Pull Request (PR) since it detects your repo is a clone of the upstream. | 6. PR can now be reviewed and merged by others. I've run into an issue with this procedure, that I'm trying to work through. -- :ar
