Add UTM to/from lat long convertor 0.9.8 #4202
Conversation
|
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
|
Failure in build 4 (
WARN: Remotes registry file missing, creating default one in /home/conan/w/cci_PR-4202/4/19007ab8-9b64-4570-b733-43dbd463959c/.conan/remotes.json |
| #define RAD_TO_DEG (180.0/M_PI) """,) | ||
|
|
||
| def package_info(self): | ||
| # Original CMakeLists.txt exports "utm_latlong_convertor::utm_latlong_convertor" target: |
There was a problem hiding this comment.
Do you have a link? I don't see anything in upstream's source code.
There was a problem hiding this comment.
Oh sorry I think I just pasted this from other recipe files...will get rid of it thanks
| self.copy(pattern="LICENSE.md", dst="licenses", | ||
| src=self._source_subfolder) | ||
|
|
||
| self.copy(pattern="UTM.h", dst="include", | ||
| src=os.path.join(self._source_subfolder, "libs", "UTM", "include", "UTM")) | ||
|
|
||
| tools.replace_in_file(os.path.join(self.package_folder, "include", "UTM.h"), | ||
| "#include \"ofConstants.h\"", "// #include \"ofConstants.h\"") | ||
| tools.replace_in_file(os.path.join(self.package_folder, "include", "UTM.h"), "#define UTM_EP2 (UTM_E2/(1-UTM_E2)) ///< e'^2", """#define UTM_EP2 (UTM_E2/(1-UTM_E2)) ///< e'^2 | ||
| #define DEG_TO_RAD (M_PI/180.0) | ||
| #define RAD_TO_DEG (180.0/M_PI) """,) |
There was a problem hiding this comment.
Shouldn't you build a lib? There are others public headers in this library, and .cpp files. But I guess it requires https://github.com/openframeworks/openFrameworks
There was a problem hiding this comment.
So, the functionality to convert from UTM to Lat Lon is just this one file called UTM.h. Rest of the files have other functionalities which are non overlapping. So I thought of making it only a header-only with that one file. The replace_in_file is just a way around that dependancy on ofConstants.h.
It would be really helpful if you have a better and a cleaner solution in mind. Thanks :)
There was a problem hiding this comment.
2 solutions:
- create a recipe for openFrameworks first, then create your recipe
- create your recipe as is, but add a TODO for full implementation of
ofxgeo(which is not only this header file), and display a warning for consumers that this recipe is just a subset ofofxgeo.
|
Failure in build 5 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions. |
Specify library name and version: utm_latlong_convertor/0.9.8
conan-center hook activated.