[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

Nicolas Chauvet  changed:

   What|Removed |Added

 Blocks|3   |


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3
[Bug 3] Tracker: Packages under review.
-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

Richard  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |INVALID

--- Comment #30 from Richard  ---
Since OCE is now in Fedora I assume this can be in Fedora as well. Closing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #29 from Richard hobbes1...@gmail.com 2014-05-27 16:07:55 CEST ---
Now that OCE is in Fedora I assume this bug can be closed?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #27 from mrcer...@gmail.com 2013-01-10 13:07:07 CET ---
Hello Richard,
Once registered, I see these links: 
http://www.salome-platform.org/downloads/current-version/DownloadDistr?platform=Sourcesversion=6.6.0
http://www.salome-platform.org/downloads/current-version/DownloadDistr?platform=Documentationversion=6.6.0
http://git.salome-platform.org/gitweb/

they seem to be static and not related to the login, but I'm not sure.

However, I sent an email to the legal list to ask for advice:
http://lists.fedoraproject.org/pipermail/legal/2013-January/002061.html

and I'll post here whatever they answer to me.

Best,
Mario

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #28 from Miloš Komarčević kmi...@gmail.com 2013-01-10 13:20:52 
CET ---
(In reply to comment #25)
 I am a user of Salome Meca on caelinux (http://www.caelinux.com/CMS/) and I
 would like to see it in Fedora too. I'm curious why you packaged geom instead
 of the original salome platform (maybe some licensing/funcionality/personal
 preference issues?). Can I help speed up the review in some way?

Thanks for the interest Mario. I admit I have not looked into licensing issues
for Salome. My motivation was to get the bare minimum in to support packaging
pythonOCC next. (I guess the same was true for Richard packaging SMESH for
FreeCAD.)

For this particular package, you can help by trying to generate some interest
upstream perhaps?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #26 from Richard hobbes1...@gmail.com 2013-01-09 15:12:54 CET ---
(In reply to comment #25)
 Hello Milos, hello Richard,
 I am a user of Salome Meca on caelinux (http://www.caelinux.com/CMS/) and I
 would like to see it in Fedora too. I'm curious why you packaged geom instead
 of the original salome platform (maybe some licensing/funcionality/personal
 preference issues?). Can I help speed up the review in some way?

I'm not sure. Has SALOME always been LGPL licensed? I don't like the fact it
requires a login to download the source though...

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

mrcer...@gmail.com changed:

   What|Removed |Added

 CC||mrcer...@gmail.com

--- Comment #25 from mrcer...@gmail.com 2013-01-07 17:20:55 CET ---
Hello Milos, hello Richard,
I am a user of Salome Meca on caelinux (http://www.caelinux.com/CMS/) and I
would like to see it in Fedora too. I'm curious why you packaged geom instead
of the original salome platform (maybe some licensing/funcionality/personal
preference issues?). Can I help speed up the review in some way?

Thanks, 

Mario

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #24 from Miloš Komarčević kmi...@gmail.com 2012-12-20 11:14:54 
CET ---
 Have you had any luck with upstream?

Nope.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #23 from Richard hobbes1...@gmail.com 2012-12-19 16:39:26 CET ---
I had a few minutes and looked through open reviews and ran across this one. I
hate to admit I had completely forgotten about it! :)

Have you had any luck with upstream?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #21 from Richard hobbes1...@gmail.com 2012-09-29 16:34:13 CEST ---
Is this review still active? I admint I haven't had much time lately but I'm
willing to help move it along.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #22 from Miloš Komarčević kmi...@gmail.com 2012-09-29 16:52:40 
CEST ---
I had no response from Fotis on upstream tickets and patches reported to him on
Sourceforge (also pinged on email), so if you're in touch with him, maybe you
can help raise some interest?

It also looks he's more focused on the WOK based version [1] going forward,
which I'm also not sure how well it'll fit with OCE?

http://salomegeometry.svn.sourceforge.net/viewvc/salomegeometry/

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

Alec Leamas leamas.a...@gmail.com changed:

   What|Removed |Added

 CC||leamas.a...@gmail.com

--- Comment #18 from Alec Leamas leamas.a...@gmail.com 2012-02-18 12:42:56 
CET ---
Looks lika a build problem, some file are not compiled. Focusing on 
Sketcher::GetLabel(), one of the undefined symbols:

Build the thing: 
$ rpmbuild -bc  ~/rpmbuild/SPECS/geom*spec 21 | tee build.log
$ pushd ~/rpmbuild/geom*

There is no defined symbol in the object files:
$ nm -gC $(find . -name \*.o) | grep  'Sketcher::GetLabel'
 U Sketcher::GetLabel()

GetLabel lives in src/Sketcher.cpp:
$ grep -r Sketcher::GetLabel src
src/Sketcher/Sketcher.cpp:TDF_Label Sketcher::GetLabel()

There is no file Sketcher.o, and Sketcher.cpp doesn't seem to be compiled:
$ find . -name Sketcher.o
$ popd
$ grep Sketcher.cpp build.log

just my two cents...

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #19 from Miloš Komarčević kmi...@gmail.com 2012-02-18 15:06:39 
CET ---
(In reply to comment #18)
 There is no file Sketcher.o, and Sketcher.cpp doesn't seem to be compiled:
 $ find . -name Sketcher.o
 $ popd
 $ grep Sketcher.cpp build.log
 
 just my two cents...

Thanks Alec, that one was actually down to globbing for Sketcher_*.cpp instead
of Sketcher*.cpp in CMakeLists.txt

Just by picking a few other ones at random, it does look like the
implementation of those methods really is missing in the source files, although
the definitions are there in the header files, so file another bug upstream.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #20 from Miloš Komarčević kmi...@gmail.com 2012-02-18 16:53:20 
CET ---
Missing method implementation upstream bug:

https://sourceforge.net/tracker/?func=detailaid=3489036group_id=214102atid=1028339

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #17 from Miloš Komarčević kmi...@gmail.com 2012-02-18 01:05:38 
CET ---
SPEC: http://kmilos.fedorapeople.org/geom.spec 
SRPM: http://kmilos.fedorapeople.org/geom-6.3.1.8-5.svn222.fc16.src.rpm

(In reply to comment #16)
 The first is undefined-non-weak-symbol which rpmlint describes as:
 $ rpmlint -I undefined-non-weak-symbol
 undefined-non-weak-symbol:
 The binary contains undefined non-weak symbols.  This may indicate improper
 linkage; check that the binary has been linked as expected.
 
 Basically, it has symbols for libraries it's not linked to.

Ok, managed to find a few missing, but the majority is still there. The symbols
seem to come from the GEOM libs themselves - no idea how to handle this, help
appreciated.

 The other, easier one, is: unused-direct-shlib-dependency
 
 The most direct method is to add -Wl,--as-needed to your linker flags, a la:
 LDFLAGS=-Wl,--as-needed;export LDFLAGS
 
 just before your cmake command.

Had this in already from your SMESH spec file.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #14 from Miloš Komarčević kmi...@gmail.com 2012-02-11 20:06:26 
CET ---
Ok, did it the pkg-config way.

SPEC: http://kmilos.fedorapeople.org/geom.spec 
SRPM: http://kmilos.fedorapeople.org/geom-6.3.1.8-4.svn222.fc16.src.rpm

$ rpmlint SRPMS/*.rpm RPMS/noarch/*.rpm RPMS/i686/*.rpm
geom.src: W: invalid-license LGPLv2 with exception
geom.src: W: invalid-url Source0: geom-6.3.1.8-svn222.tar.gz
geom-doc.noarch: W: invalid-license LGPLv2 with exception
geom.i686: W: invalid-license LGPLv2 with exception
geom.i686: E: incorrect-fsf-address
/usr/share/doc/geom-6.3.1.8/LICENCE.lgpl.txt
geom-debuginfo.i686: W: invalid-license LGPLv2 with exception
geom-devel.i686: W: invalid-license LGPLv2 with exception
geom-devel.i686: W: no-documentation
6 packages and 0 specfiles checked; 1 errors, 9 warnings.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #12 from Miloš Komarčević kmi...@gmail.com 2012-02-10 23:51:06 
CET ---
Sigh. I really hoped I could pull a fast one on that, and not pull in pkgconfig
as a BR as well...

Also, just out of curiosity, I don't get why libXmu and freeimage are needed at
build time? They're not included anywhere in the code, shouldn't dynamic
linking of OCE libs take care of the rest?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #13 from Richard hobbes1...@gmail.com 2012-02-11 02:45:24 CET ---
(In reply to comment #12)
 Sigh. I really hoped I could pull a fast one on that, and not pull in 
 pkgconfig
 as a BR as well...

Eh... :) Not really a big deal. Another option would be to use FIND_FILE(...
and FIND_LIBRARY(... to do things manually but the pkg-config module does make
things easy.


 Also, just out of curiosity, I don't get why libXmu and freeimage are needed 
 at
 build time? They're not included anywhere in the code, shouldn't dynamic
 linking of OCE libs take care of the rest?

Not sure... I'd have to investigate. Either their cmake config is requiring it
even if it's not being used, or it's a requirement of one of the other main
requirements?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #11 from Richard hobbes1...@gmail.com 2012-02-08 18:05:17 CET ---
(In reply to comment #10)
 Note: Probably didn't do the proper thing of finding the lib3ds the CMake way.
 Got lucky with the way lib3ds is packaged and included in GEOM source, so the
 BuildRequires suffices, hope that's ok for now.

I can help there. It looks like lib3ds-devel provides a pkgconf config file so
something like this should work:

INCLUDE(FindPkgConfig)
IF(USE_EXTERNAL_LIB3DS)
   # The last argument may need to be lib3ds depending on the pkgconf
config.
   PKG_CHECK_MODULES(LIB3DS REQUIRED 3ds)
   INCLUDE_DIRECTORIES(SYSTEM ${LIB3DS_INCLUDE_DIRS})
ENDIF()
---

And then the following:
TARGET_LINK_LIBRARIES(Exchange3DS TKernel TKBRep TKMath TKMesh TKV3d TKTopAlgo
3ds)
would become:
TARGET_LINK_LIBRARIES(Exchange3DS TKernel TKBRep TKMath TKMesh TKV3d TKTopAlgo
${LIB3DS_LIBRARIES})
---

Also, if you use the option as shown above you'll need to add that to you
cmake options:
%cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo \
   -DUSE_EXTERNAL_LIB3DS=TRUE \
   ..
---

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #5 from Richard hobbes1...@gmail.com 2012-02-05 15:12:47 CET ---
Have you been able to get any clarification from upstream on the license?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #6 from Miloš Komarčević kmi...@gmail.com 2012-02-05 15:31:17 CET 
---
(In reply to comment #5)
 Have you been able to get any clarification from upstream on the license?

I have included this request for clarification as a comment on

http://sourceforge.net/tracker/?func=detailaid=3477860group_id=214102atid=1028339

So far, no word on any of the bugs filed upstream :(

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

rc040...@freenet.de changed:

   What|Removed |Added

 CC||rc040...@freenet.de

--- Comment #7 from rc040...@freenet.de 2012-02-05 17:41:49 CET ---
(In reply to comment #4)
 Although he may be stuck with the verbage since
 this is effectivly a fork of the original, right?

Well, the files in geom-6.3.1.8/src/Exchange3DS/lib3ds/ are not a fork, they
are a bundled copy of lib3ds-1.3.0 (all files are bit-wise identical).

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #8 from Miloš Komarčević kmi...@gmail.com 2012-02-05 17:53:01 CET 
---
(In reply to comment #7)
 Well, the files in geom-6.3.1.8/src/Exchange3DS/lib3ds/ are not a fork, they
 are a bundled copy of lib3ds-1.3.0 (all files are bit-wise identical).

Ah, thanks fro checking. That makes things a bit more complicated, as this
probably needs to be unbundled and packaged separately? Unless the lib3ds
project is being merged into GEOM?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #10 from Miloš Komarčević kmi...@gmail.com 2012-02-06 00:35:39 
CET ---
SPEC: http://kmilos.fedorapeople.org/geom.spec 
SRPM: http://kmilos.fedorapeople.org/geom-6.3.1.8-3.svn222.fc16.src.rpm

$ rpmlint SRPMS/*.rpm 
geom.src: W: invalid-license LGPLv2 with exception
geom.src: W: invalid-url Source0: geom-6.3.1.8-svn222.tar.gz
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint RPMS/i686/*.rpm 
geom.i686: W: invalid-license LGPLv2 with exception
geom.i686: E: incorrect-fsf-address
/usr/share/doc/geom-6.3.1.8/LICENCE.lgpl.txt
geom-debuginfo.i686: W: invalid-license LGPLv2 with exception
geom-devel.i686: W: invalid-license LGPLv2 with exception
geom-devel.i686: W: no-documentation
3 packages and 0 specfiles checked; 1 errors, 4 warnings.

$ rpmlint RPMS/noarch/*.rpm 
geom-doc.noarch: W: invalid-license LGPLv2 with exception
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Latest version unbundles lib3ds and also corrects the preprocessing
INCLUDE_PATH in the Doxygen template (Richard, you might want to check SMESH
too).

Note: Probably didn't do the proper thing of finding the lib3ds the CMake way.
Got lucky with the way lib3ds is packaged and included in GEOM source, so the
BuildRequires suffices, hope that's ok for now.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #2 from Richard hobbes1...@gmail.com 2012-01-24 16:53:05 CET ---
1. I'm assuming you didn't try to build in mock. I found I needed to add the
following BR's to get a successful build:

BuildRequires: libXmu-devel
BuildRequires: freeimage-devel

2. Have you reported the incorrect FSF address upstream?

3. Disclaimer: I'm not a license expert :)

Can you show me what/where in the source made you decide that the license was
LGPLv2+? I couldn't find any meaningful occurence of or later. 

There were a one directory of LGPLv2+ licensed files, src/Exchange3DS, but
all the other files seemed to be LGPLv2. 

Also, since this links against OCE it should have the same statement I added to
smech. Just append with exception to the end of whatever license we figure
out.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #3 from Miloš Komarčević kmi...@gmail.com 2012-01-24 20:16:11 CET 
---
I thought the spec might look somewhat familiar. ;)

Oops. You're right, did not build in mock (running low on space on my machine).
Added those BRs now.

 2. Have you reported the incorrect FSF address upstream?

http://sourceforge.net/tracker/?func=detailaid=3477860group_id=214102atid=1028339

(Have also opened issues related to the patches.)

 Can you show me what/where in the source made you decide that the license was
 LGPLv2+? I couldn't find any meaningful occurence of or later.

Ah, was confused myself. Most source files do still contain the word either
which threw me off, but it looks like or later... was removed, so it wasn't
clear cut to me at that moment. We could presume the author(s) intentionally
removed the or later... bit from standard notice [1] but forgot to correct
the sentence for grammar? Maybe I should contact the upstream to clarify/fix
this as well?

I've put it as LGPLv2 with exceptions for now.

SPEC: http://kmilos.fedorapeople.org/geom.spec 
SRPM: http://kmilos.fedorapeople.org/geom-6.3.1.8-2.svn222.fc16.src.rpm

[1] http://www.gnu.org/licenses/lgpl-2.1.html#TOC4

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

--- Comment #4 from Richard hobbes1...@gmail.com 2012-01-24 20:54:34 CET ---
(In reply to comment #3)
  Can you show me what/where in the source made you decide that the license 
  was
  LGPLv2+? I couldn't find any meaningful occurence of or later.
 
 Ah, was confused myself. Most source files do still contain the word either
 which threw me off, but it looks like or later... was removed, so it wasn't
 clear cut to me at that moment. We could presume the author(s) intentionally
 removed the or later... bit from standard notice [1] but forgot to correct
 the sentence for grammar? Maybe I should contact the upstream to clarify/fix
 this as well?
 
 I've put it as LGPLv2 with exceptions for now.

I would check with the author on what his intentions were but even if he tells
you it's LGPLv2 (or later), he still needs to make sure the license and header
files reflect this properly. Although he may be stuck with the verbage since
this is effectivly a fork of the original, right?

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

[Bug 2150] Review request: geom - SALOME Geometry Module

[RPM Fusion Bugzilla]

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2150

Richard hobbes1...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||hobbes1...@gmail.com
 Blocks||3
 AssignedTo|rpmfusion-package-review@rp |hobbes1...@gmail.com
   |mfusion.org |

--- Comment #1 from Richard hobbes1...@gmail.com 2012-01-23 21:53:39 CET ---
FYI, You forgot to block bug 2 (the new review request tracker) but since I'm
taking it, don't worry about doing it now.

Spec looks good :)

I'll hopefully have time to work on the full review today or tomorrow.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.