Skip to content
This repository was archived by the owner on Jan 14, 2023. It is now read-only.

Commit 8c3748c

Browse files
committed
[rosjava_messages] historical notes for devs
1 parent 7949d7e commit 8c3748c

File tree

2 files changed

+29
-2
lines changed

2 files changed

+29
-2
lines changed

Readme-Developers.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# RosJava Messages
2+
3+
## History
4+
5+
Some history for the annals of time and also for future devs who are wondering about
6+
the boundaries defining the choice of message packages that form the conglomeration
7+
that is rosjava messages.
8+
9+
Several approaches were tried/tested, discarded until we have what is here.
10+
11+
**Scan rosdistro to discover all possible message packages**
12+
13+
Experimental packages often appear/disappear and this triggered too much manual intervention to maintain. There is also far too many packages that have code with it too. This is awkward for rosjava to depend on - I prefer message only packages and have encouraged upstream folk to shift their messages to message only packages in certain instances.
14+
15+
*Used for hydro, but dropped for indigo*
16+
17+
**Minimal set of messages directly used by rosjava/android**
18+
19+
Easiest to maintain, but not so useful for users - they have to then go off and generate many message packages themselves, even if only using some of the more common packages.
20+
21+
*Used for indigo, but dropped shortly afterwards*
22+
23+
**Minimal set and finite set of stable, popular packages**
24+
25+
Relatively easy to maintain and important for maven repo users since they will not want to go and directly build message packages themselves. If I get a request, I usually add to the list to support these users.
26+
27+
*Used in indigo*

Readme.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ This package is a generator of rosjava message artifacts for core ros messages.
44

55
### Adding Packages
66

7-
If you would like to add a package to this list, then please modify
8-
CMakeLists.txt and package.xml appropriately, then send us a pull request!
7+
If you would like to add a package to this list, please modify
8+
both CMakeLists.txt, package.xml appropriately and send us a pull request!
99

1010
### More Information
1111

0 commit comments

Comments
 (0)