mqttx-cli: init at 1.11.0

[pineapplehunter]

Description of changes

Adds mqttx-cli package.

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[pineapplehunter]

added version test

[pineapplehunter]

I just realized that the binary name conflicts with mqttx desktop app.
I'll check what others are doing in this case.

[pineapplehunter]

Looking at other repos such as AUR, they don't seem to have mqttx-cli in them.
I can think of three solutions to the conflicting binary name problem.

1. Let the user install only one package
2. Change the desktop app binary to mqttx-desktop or something similar
3. Change the cli app binary to mqtttx-cli or something similar

The first option does not seem good because I can see some users wanting both packages at the same time. The second option seems good IMO but is a breaking change for the users currently using MQTTX desktop app. The third option seems to be the easiest.

I'll change this cli app name to mqttx-cli for now.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/4589

[pineapplehunter]

Updated package to 1.11.0 and fixed the update script

[FliegendeWurst]

Suggested change

	cd cli
	export HOME="$(mktemp -d)"
	cd cli

Might fix the build failure.

[pineapplehunter]

I am not able to reproduce the build failure on my x86_64-darwin machine...
I know that hydra is using rosetta on aarch64-darwin to build x86_64-darwin packages, so it may be an issue with that. I will try building it on an aarch64-darwin machine when I have the time.

[FliegendeWurst]

Did you rebase the PR on master before building? (that is almost what ofborg does)

[pineapplehunter]

I ran nixpkgs-review on a x86_64-darwin machine.

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 337730

---

x86_64-darwin

✅ 1 package built:

• mqttx-cli

[FliegendeWurst]

@ofborg build mqttx-cli

We can always try again