-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename package from Proj4.jl to Proj.jl #26
Comments
I'm not sure about capitalization, but other than that 👍 To quote the official word on naming (https://proj4.org/news.html#versioning-and-naming):
So maybe just Proj.jl? |
Ha, yeah, there they name there binary and library by their lowercase filename, but the project they call PROJ. Why do you think it is better not to follow their capitalisation of the project? Not sure why they use uppercase since it doesn't seem to be an acronym. But yeah, Proj is a bit nicer to type. Doesn't go as well with GEOS and GDAL though ;) |
Well, the julia wrapper is just a wrapper for libproj, not the proj command line binary? So it's not the Having said all that, I'll defer to the current users :-) |
Yeah I see what you mean, but I believe most wrappers of We do distribute the And it might make sense to document how to use them like done for GDAL.jl: https://github.com/JuliaGeo/GDAL.jl/#using-the-gdal-and-ogr-utilities Of course in most serious applications it's better not to shell out, but it can be convenient when converting a command line script. BTW currently it is |
The Lib*.jl names were due to me in an earlier period. The modern convention for package names is to move away from them, so I'll be happy for the renaming to happen. |
Currently renaming without registering a new package does not seem possible yet. |
Actually on further consideration I agree with Chris that Proj.jl is probably a nicer new package name. It's building PROJ 7 right now over at JuliaPackaging/Yggdrasil#966. I'm currently thinking to keep the PROJ_jll compat in this package to max 6, and in the new package 7 and higher. I think that means they cannot be used at the same time though. But I believe that for GDAL builds, we can be flexible and let it pick either. |
I still like Proj.jl 👍 I'm not sure you can just rename and keep the UUID, I think that may cause issues in I'd be tempted to rename the repo on github so that issue history is preserved in the main project, then reverse-fork it back to the old name if necessary to maintain the old API during the transition. (Might be worth testing that this actually works though with a dummy project before committing to it!) |
Hmm interesting. From JuliaRegistries/General#2770 I gathered we have to change the UUID. We could of course do that in this repo, and just add commits that change the API. Perhaps we can start working in a |
Or perhaps we should just start developing the new API as a second API on master, holding off the breaking changes (removing the old API, changing the UUID) until the end. That may be a bit easier. |
See #23 (comment) and comment below.
The text was updated successfully, but these errors were encountered: