Bug 253020 - Review Request: animorph - 3D Animation and Morph Library
Review Request: animorph - 3D Animation and Morph Library
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mamoru TASAKA
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 253022
  Show dependency treegraph
 
Reported: 2007-08-16 13:18 EDT by Nicolas Chauvet (kwizart)
Modified: 2007-11-30 17:12 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-09-13 08:26:46 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mtasaka: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Nicolas Chauvet (kwizart) 2007-08-16 13:18:20 EDT
Spec URL:
http://kwizart.free.fr/fedora/6/testing/makehuman/animorph.spec
SRPM URL:
http://kwizart.free.fr/fedora/6/testing/makehuman/animorph-0.2-1.kwizart.fc6.src.rpm
Description: Libraries for animorph
Comment 1 Mamoru TASAKA 2007-08-26 09:08:20 EDT
Well, as this is rather a small package, there are
not so many items to be checked...

For 0.2-1:
* Redundant BuildRequires
  - glibc-headers in minimal build environ

* License
  - The following source files are under GPLv2+
-----------------------------------------------
./include/animorph/PoseSemiTarget.h
./include/animorph/PoseTarget.h
./src/PoseSemiTarget.cpp
./src/PoseTarget.cpp
------------------------------------------------
    Change license tag to GPLv2+
    (However, I guess the author intended to release
     this under LGPLv2+. So after/before this package
     is approved, would you contact with upstream?)
Comment 2 Nicolas Chauvet (kwizart) 2007-08-26 17:27:22 EDT
Actually the COPYING says LGPLv2+
e-mail send upstream about the license question...
I hope an answear to have this state clearly...
Comment 3 Mamoru TASAKA 2007-09-04 15:01:27 EDT
Any response from upstream?
Comment 4 Nicolas Chauvet (kwizart) 2007-09-04 18:11:09 EDT
not any answear for now.
I will send it again...

Comment 5 Nicolas Chauvet (kwizart) 2007-09-10 19:17:49 EDT
What policy should we follow in this case (still no answear from upstream)

Should we choose the more restrictive one  The one which is written in the
source code headers ?

If animoph is GPL that implies that every app that link against it to be GPL
also...So this change nothing about makehuman. I think that thoses libs was
certainly taken out of makehumain to be buildable externally.. So they only
changed the COPYING while doing make dist...



Comment 6 Mamoru TASAKA 2007-09-10 21:44:57 EDT
(In reply to comment #5)
> Should we choose the more restrictive one  The one which is written in the
> source code headers ?

We must choose the "actual" license (i.e. we must judge the license from
source code). In this case, the license is GPLv2+.
Comment 7 Nicolas Chauvet (kwizart) 2007-09-12 08:02:54 EDT
Should I remove the COPYING as it conflict with what is said within the source
code ?
Does I need also to add thr right COPYING file ?
Comment 8 Mamoru TASAKA 2007-09-12 08:41:55 EDT
The original COPYING file can be removed and I don't think
we have to add the right license text.
Comment 9 Nicolas Chauvet (kwizart) 2007-09-12 09:00:06 EDT
Spec URL:
http://kwizart.free.fr/fedora/6/testing/makehuman/animorph.spec
SRPM URL:
http://kwizart.free.fr/fedora/6/testing/makehuman/animorph-0.2-2.kwizart.fc6.src.rpm
Description: 3D Animation and Morph Library

%changelog
- Change license to GPLv2+ as said in the source code
- Remove BR glibc-headers
- Change summary (taken from animorph.pc)
Comment 10 Mamoru TASAKA 2007-09-12 10:14:21 EDT
Okay!

-----------------------------------------------------
     This pacakge (animorph) is APPROVED by me
-----------------------------------------------------
Comment 11 Nicolas Chauvet (kwizart) 2007-09-12 16:27:57 EDT
New Package CVS Request
=======================
Package Name:      animorph
Short Description: 3D Animation and Morph Library
Owners:            kwizart
Branches:          devel FC-6 F-7 EL-4 EL-5
InitialCC:         <empty>
Commits by cvsextras: yes
Comment 12 Kevin Fenzi 2007-09-12 23:42:16 EDT
cvs done.

Note You need to log in before you can comment on or make changes to this bug.