Shorten binary names? #107
Closed
gavanderhoorn
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
I'm ok with your proposal. But we should add a Then there's also the question of whether the micro controller uses |
Beta Was this translation helpful? Give feedback.
1 reply
-
See #109 for a PR implementing the above proposal. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We currently use names such as
motoros2_yrc1000_galactic.out
.While clear, those names are long, and may not work on all controllers we (want) to support. It looks like at least the DX2 has trouble loading such binaries (#49 (comment)), and FS is also expected not to be compatible.
I'd suggest we consider shortening the names, such that it's still clear what the binary is for (application name), and what it supports (controller series and ROS 2 version).
Within ROS 2 this is common practice: all ROS 2 versions have a codename, which is a single word identifier which is often used instead of the full name. Humble Hawksbill is referred to as
humble
for instance.ROS 2 CI and build jobs also use a scheme to abbreviate the names: Hbin_ujv8_uJv8 shows all the
H
umblebin
ary jobs targettingu
buntuj
azzy running on ARMv8
(ie:arm64
).This works because ROS 2 release and code names are all named alphabetically and uniquely, so the first character of a codename can be used to sort entries and is 'guaranteed' to be unique (in quotes, as there are only 26 letters in the alphabet of course).
I'm not proposing we obfuscate the names that much of course.
But what about
mr2_yrc_g.out
? That would beM
otoR
OS2
forYRC
1000 supportingG
alactic.And
mr2_dx2_h.out
would beM
otoR
OS2
forDX2
00 supportingH
umble.And
mr2_fs_i.out
would beM
otoR
OS2
forFS
100 supportingI
ron.Note that the names of the distribution
.zip
s can stay as they are, so there the unabbreviated names would still be used. This should help avoid confusion.Beta Was this translation helpful? Give feedback.
All reactions