Bug 566977

Summary:	Review Request: libSNL - library for manipulating NURBS surfaces
Product:	[Fedora] Fedora	Reporter:	Manfred Spraul <manfred>
Component:	Package Review	Assignee:	Dan Horák <dan>
Status:	CLOSED DUPLICATE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	rawhide	CC:	dan, fedora-package-review, msuchy, notting
Target Milestone:	---	Flags:	dan: fedora-review?
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-12-15 22:56:34 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	566974

Description Manfred Spraul 2010-02-20 20:43:33 UTC

libSNL is a library for manipulating NURBS surfaces.

Base packages:
- exists as project on sourceforge
- http://sourceforge.net/projects/libsnl/
- no updates in the last 3 years
- no files were ever released, only an svn tree exists.
- there is no documentation on the package
- C++ language
- license: GPLv2+

rpm package:
- the svn snapshot from Feb 13th 2010 is used for the package
- two rpms are created: one with the dynamic library, one with the header files.
- the header files are stored under /usr/include/SNL

Comment 1 Manfred Spraul 2010-02-21 16:22:56 UTC

ldconfig in %post and %postun were missing.

new spec file:
http://sourceforge.net/projects/calculix-rpm/files/fedora/libSNL.spec/download

new SRPM:
http://sourceforge.net/projects/calculix-rpm/files/fedora/libSNL-0.2-2.fc12.src.rpm/download

Comment 2 Dan Horák 2010-02-23 09:18:04 UTC

Again few notes:
- preferred way to obtain a snapshot is to use "svn export" (and use the svn revision id because it uniquely identifies the snapshot) and we have a wrapper that simplifies the process in the fedora-packager package, its usage is
"fedora-getsvn libsnl https://libsnl.svn.sourceforge.net/svnroot/libsnl/trunk 18"
- the version/release tags should use the "snapshot package" versioning - see https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages
- would be useful if the maintainance would be coordinated with other distros carrying/wanting to carry Calculix and eventually take over the upstream development/maintainance, I see subtle difference between the upstream libSNL code and the version packaged in cgx sources

Comment 3 Miroslav Suchý 2012-12-11 23:04:34 UTC

Ping? Any progress here? Or we can close this review?

Comment 4 Manfred Spraul 2012-12-15 22:56:34 UTC

I've closed the review request due to lack of time to finish it.
I'll try to keep the package up to date, i.e. merge any changes from upstream.

With regards to the existence of this package: I'm not sure if it makes sense to create a libSNL package at all: As Dan noticed, there are differences between the version in the cgx sources and the version on sourceforge, and unlike cgx (latest release Oct 2012) the version on sourceforge is unmaintained (last update Apr 2007).
Thus an exception to use the version within cgx might make sense - especially since cgx seems to be the only user of libSNL.

*** This bug has been marked as a duplicate of bug 558535 ***