From: Alexander P. <ale...@gm...> - 2021-05-11 17:28:21
|
Hello Stefan, I hadn't deleted the previous srpms and as a result I spent an hour building and rebuilding the wrong one, wondering why I'm not getting the documentation built… Everything seems fine, I think you can increment that last zero to one if you want. Thank you for all your work! Best regards, A. On Tue, May 11, 2021 at 4:47 PM Stefan Gerlach <ste...@un...> wrote: > > Hi Alexander, > > thanks. I fixed the GNUInstallDirs, made the doxygen doc build per default and build opj2dat with the dynamic lib if static is disabled in the > latest commit. I hope that fixes all issues :-) > > best > Stefan > > > On 11/05/2021 16:09, Alexander Ploumistos wrote: > > Hi Stefan, > > > > I had to add "include(GNUInstallDirs)" before "install" in the shared > > library section, e.g. > > > > --- a/CMakeLists.txt 2021-05-11 14:20:57.000000000 +0200 > > +++ b/CMakeLists.txt 2021-05-11 15:52:50.279765255 +0200 > > @@ -86,6 +86,7 @@ > > VERSION ${PROJECT_VERSION} > > SOVERSION ${PROJECT_VERSION_MAJOR} > > ) > > + include(GNUInstallDirs) > > install( TARGETS origin_shared > > LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} > > ) > > > > otherwise the shared object files end up under /usr/lib even on 64-bit > > arches. I suppose the static library is also affected by this. Can the > > include statement go to the beginning of the file? > > > > > > The HTML documentation is no longer created, is there an extra > > configuration/build option I need to specify or did it get eaten up > > with the latest changes? > > > > > > I don't know anyone using opj2dat by itself, if anyone files a bug > > report, I'll need to find a way to build it along with the shared > > libraries, but for now it doesn't matter. > > > > > > Best regards, > > A. > > > > > > On Tue, May 11, 2021 at 2:34 PM Stefan Gerlach > > <ste...@un...> wrote: > >> > >> Hello Alexander, > >> > >> i tried to fix the issues with liborigin in the upstream code. Let me know if there are still issues. > >> If i get the discussion correctly SciDAVis will also use the upstream version now or at least sync with upstream. Thanks for resolving these issues. > >> > >> Best regards, > >> Stefan > >> > >> On 11/05/2021 12:40, Alexander Ploumistos wrote: > >>> Hello Stefan, > >>> > >>> Thank you for taking care of this. It would be great to get a 3.0.1 > >>> release, but not just yet; when I packaged the latest snapshot with > >>> the commits from SciDAVis, I came across a couple of issues: > >>> https://github.com/highperformancecoder/scidavis/issues/218 > >>> They are not that important, but I think it would be better to get rid > >>> of them before the next release. Also, it might be a good idea to > >>> chime in on the issue or otherwise get in touch with the SciDAVis > >>> people, so as to avoid doing the work twice. If you get the chance to > >>> look at these, commit the changes and they can pick them up > >>> downstream. If they manage to get to them first, they should push them > >>> upstream. > >>> The later half of this discussion might also be of interest to you as well: > >>> https://github.com/highperformancecoder/scidavis/discussions/212 > >>> > >>> Best regards, > >>> A. > >>> > >>> On Tue, May 11, 2021 at 12:25 PM Stefan Gerlach > >>> <ste...@un...> wrote: > >>>> > >>>> Hello Alexander, > >>>> > >>>> i updated upstream liborigin with the latest patches from SciDAVis. I could release a new version 3.0.1 if that's a good idea. > >>>> LabPlot now uses the system installed liborigin 3 if available. > >>>> > >>>> best wishes > >>>> Stefan > >>>> > >>>>> You can see the changes committed to SciDAVis's liborigin submodule here: > >>>>> https://github.com/SciDAVis/liborigin/commits/ > >>>>> There are some fixes and cleanups and I don't think there's anything > >>>>> too SciDAVis-centric among them. Some of them will make my life more > >>>>> difficult as I'll probably need to refactor a few Fedora-specific > >>>>> workarounds with regard to cmake… > >>>>> > >>>>> From the point of view of a distribution package maintainer it would > >>>>> be great if all the work happened first in liborigin upstream or if > >>>>> any changes made downstream were pushed upstream (within a reasonable > >>>>> timeframe), so as to avoid forking and fragmentation. While git > >>>>> submodules are appealing, at least to consumers of projects, I think > >>>>> they tend to complicate things. > >>>>> > >>>>> Once you get a chance to look at the code, let me know what you think. > >>>>> > >>>>> > >>>>> Thank you and have a nice weekend, > >>>>> A. > >>>>> > >>>>> > >>>>> On Sat, May 8, 2021 at 11:09 AM Stefan Gerlach > >>>>> <ste...@un...> wrote: > >>>>>> > >>>>>> Hello Alexander, > >>>>>> > >>>>>> i'm not working on SciDAVis but i also would like to see the changes incorporated in liborigin. > >>>>>> Missing updates of liborigin were the reason why we use a bundled version of liborigin in LabPlot. If nobody cares, i can take a look at > >>>>>> SCiDAVis liborigin and try to update liborigin. If that works, we can use the official liborigin in LabPlot. > >>>>>> > >>>>>> best wishes > >>>>>> Stefan > >>>>>> > >>>>>> On 07/05/2021 23:04, Alexander Ploumistos wrote: > >>>>>>> Hello Stefan, Miquel, > >>>>>>> > >>>>>>> I was about to update the Fedora packages of SciDAVis and its > >>>>>>> dependencies for which I am the maintainer and I realized that the > >>>>>>> changes committed to the SciDAVis branch of liborigin have not been > >>>>>>> incorporated yet into the "official" repository. > >>>>>>> > >>>>>>> In theory, the LabPlot should also depend on liborigin, but looking at > >>>>>>> the spec file, it seems that at some point one of the maintainers > >>>>>>> switched to the bundled version. Nevertheless, I'd like to abide by > >>>>>>> Fedora's rules and maintain a separate package - if possible - which I > >>>>>>> think it should be. > >>>>>>> > >>>>>>> Since the two of you are contributing to all of the above projects, I > >>>>>>> was wondering if you intend to merge the changes from SciDAVis to > >>>>>>> liborigin, so that there is one upstream source for everything. > >>>>>>> > >>>>>>> > >>>>>>> Best regards, > >>>>>>> A. > >>>>>>> > >>>>>>> > >>>>>>> _______________________________________________ > >>>>>>> Liborigin-devel mailing list > >>>>>>> Lib...@li... > >>>>>>> https://lists.sourceforge.net/lists/listinfo/liborigin-devel > >>>> |