search

dartsim / dart

DART: Dynamic Animation and Robotics Toolkit
http://dartsim.github.io/
BSD 2-Clause "Simplified" License
903 stars 286 forks source link

DART needs a new name #479

Open karenliu opened 9 years ago

karenliu commented 9 years ago

Dear DART community,

It's been exciting to see how much DART grow in the past two years thanks to our devoted users and contributors. As DART becomes increasingly popular, we started to receive some requests of name change from other software owners who use similar names. The most similar one is the DARTS rigid body simulator from JPL. To avoid potential confusion down the road, we have made a painful but necessary decision to change the name effective on the next major version release. @jslee02, @mxgrey, and myself have come up with a short list of candidates which can be found here. We would love to hear your thoughts before we make the final decision. Thank you for your support and interest in DART.

Karen Liu

psigen commented 6 years ago

If the name of the package in CMake is dartsim, then isn't the usual convention for the debian package to be libdartsim and libdartsim-dev, etc?

j-rivero commented 6 years ago

AFAIK, there is no convention or policy in Debian with respect to how the cmake modules needs to be named with respect to the software name and I'm pretty sure that the lib* package names have more todo with the source package name than with any other aspect related to the build system.

The decision of keeping the package source name and cmake module somehow in sync respond to arbitrary criteria. Michael has described some of the reasoning that can lead to a decision but nothing is restricted or suggested.

@j-rivero in my comment #479 (comment) I noticed a problem with find_package(DART), so I think we would also want to rename the cmake config file. Would that be ok?

I see no problem with that.

jslee02 commented 6 years ago

@j-rivero Thank you for the confirmation! I have no objection changing the debian package name to libdartsim#-* and CMake package name to DARTSim. I think it would be good to have both set of libdart#-*/DART and libdartsim#-*/DARTSim for DART 6.X (X >= 4) and remove libdart#-*/DART in DART 7. If everyone agree with this, then I'll make that change.

Also, I would prefer DARTSim over DartSim but am open to hearing if DartSim or any other name would make more sense.

mxgrey commented 6 years ago

The capitalization is tricky, and it's important since cmake is capital-sensitive when it comes to package names.

I see the logic behind DARTSim, but I'm not sure if I like it, because it feels like a mix of styles instead of one consistent style. Again, the reasoning makes sense, so I'm not necessarily against it, but I might be more inclined to use all lowercase dartsim instead to have a less complicated pattern to remember.