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

amqptools: init #25317

Closed
wants to merge 1 commit into from
Closed

amqptools: init #25317

wants to merge 1 commit into from

Conversation

apeschar
Copy link
Contributor

@apeschar apeschar commented Apr 29, 2017

Motivation for this change

Add amqptools package. It's useful for testing servers such as RabbitMQ.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@apeschar, thanks for your PR! By analyzing the history of the files in this pull request, we identified @zimbatm, @zraexy and @vcunat to be potential reviewers.

@joachifm
Copy link
Contributor

Please add a version string of some sort to the name. I notice this hasn't been updated since 2012 and has no proper release. Does that mean the software is "complete"?

@apeschar
Copy link
Contributor Author

apeschar commented May 3, 2017

This pull request might have been a bit premature. Although the software works and is "complete", similar tools are included in the rabbitmq-c package, and that is actually maintained. So I don't think this should be merged, after all.

@joachifm, thanks for your feedback, and sorry for wasting your time. I'll try to make the next pull request a bit more useful.

@apeschar apeschar closed this May 3, 2017
@joachifm
Copy link
Contributor

joachifm commented May 3, 2017

@apeschar no problem :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants