Since very recently, you can now use: (string-append "CC=" ,(cc-for-target)) instead of the let binding and conditional.
Toggle quote (7 lines)> + "INSTALL=install"> + (string-append "PREFIX=" %output)))))> + (inputs> + `(("libusb-compat" ,libusb-compat)> + ("readline" ,readline)))> + (synopsis "Free debugger for use with MSP430 MCUs")
No need to mention free here, since everything in Guix is freesoftware. The GitHub tagline is pretty good: Debugging tool for MSP430 MCUs
Toggle quote (8 lines)> + (description "MspDebug supports FET430UIF, eZ430, RF2500 and Olimex> +MSP430-JTAG-TINY programmers, as well as many other compatible> +devices. It can be used as a proxy for gdb or as an independent> +debugger with support for programming, disassembly and reverse> +engineering.")> + (home-page "https://github.com/dlbeer/mspdebug")> + (license gpl2)))
This should be 'gpl2+', since the source files say "GPL 2 or any laterversion". Can you send an updated patch? Thanks in advance!
Try to keep lines within 80 characters. Have you heard of our lord & saviour ‘guix lint’? It's nitpicking like this, automated. There's a more ‘declarative’ way to disable tests. Add a short note why: `(#:tests? #f ; no test suite #:phases (modify-phases %standard-phases (delete 'configure))) ; no configure script Kind regards, T G-R