raspberrypi / picotool

BSD 3-Clause "New" or "Revised" License
555 stars 95 forks source link

No rule to install `picotool` binary #4

Closed ggardet closed 2 years ago

ggardet commented 3 years ago

All the cmake system is in place except for the installation of picotool to /usr/bin target. When make install is run, we get:

make: *** No rule to make target 'install'.  Stop.
kilograham commented 3 years ago

good idea; that would make sense

aallan commented 3 years ago

With respect to raspberrypi/pico-feedback#48 picotool should install into /usr/local/bin and not /usr/bin under macOS. Mucking about with /usr/bin on a Mac is problematic.

ggardet commented 3 years ago

With respect to raspberrypi/pico-feedback#48 picotool should install into /usr/local/bin and not /usr/bin under macOS. Mucking about with /usr/bin on a Mac is problematic.

It should be something like %{prefix}/bin anyway

lurch commented 2 years ago

Fixed by https://github.com/raspberrypi/picotool/commit/ce769705338cb57abc57d1a586983abe3a5e5199 ?

ggardet commented 2 years ago

Fixed by ce76970 ?

Likely, but I did not try it yet.

ggardet commented 2 years ago

I backported ce76970 to version 1.0.1 and I confirm this fixes this issue.

lurch commented 2 years ago

Why did you need to backport, instead of just using the latest develop version?

ggardet commented 2 years ago

I use tagged/released versions only.