Bug 427411

Summary: Review Request: common-lisp-controller - Common Lisp source and compiler manager
Product: [Fedora] Fedora Reporter: Anthony Green <green>
Component: Package ReviewAssignee: Michel Alexandre Salim <michel>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting, pbrobinson, rdieter
Target Milestone: ---Flags: michel: fedora‑review+
limburgher: fedora‑cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-11 10:51:33 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On: 427391    
Bug Blocks:    

Description Anthony Green 2008-01-03 13:01:37 EST
Spec URL: http://spindazzle.org/Fedora/common-lisp-controller.spec
SRPM URL: http://spindazzle.org/Fedora/common-lisp-controller-6.12-3.fc8.src.rpm
Description: 
This package helps installing Common Lisp sources and compilers.
It creates a user-specific cache of compiled objects. When a library
or an implementation is upgraded, all compiled objects in the cache
are flushed. It also provides tools to recompile all libraries and to
generate Fedora packages from asdf-install packages.
Comment 1 Michel Alexandre Salim 2008-05-28 22:16:37 EDT
Ready for approval, contigent on three things being fixed (listed below):
- remove tabs
- check with legal on licensing (looks more liberal than LGPL so I'm personally
OK with it)
- include debian/copyright

MUST

• rpmlint:
  - mixed use of space and tabs (trivial)
  
• package name: OK
• spec file name: OK
• package guideline-compliant: OK
• license complies with guidelines
  LLGPL not in pre-approved list. Should legal take a look at it?

• license field accurate: OK
• license file not deleted: FAIL
  please include debian/copyright file in %doc
• spec in US English: OK
• spec legible: OK
• source matches upstream: OK
• builds under >= 1 archs, others excluded: noarch, OK
• build dependencies complete: OK
• own all directories: OK
• no dupes in %files: OK
• permission: OK
• %clean RPM_BUILD_ROOT: OK
• macros used consistently: OK
• Package contains code: OK
• clean buildroot before install: OK
• filenames UTF-8: OK

SHOULD
• package build in mock on all architectures: OK
• package functioned as described: suggestion on how to test?
• scriplets are sane: OK
• require package not files: OK
Comment 2 Anthony Green 2008-07-06 09:57:03 EDT
(In reply to comment #1)
> Ready for approval, contigent on three things being fixed (listed below):
> - remove tabs
> - check with legal on licensing (looks more liberal than LGPL so I'm personally
> OK with it)
> - include debian/copyright
>


Thanks.  I've just sent a note to fedora-legal-list.

AG
Comment 3 Anthony Green 2008-07-07 11:42:42 EDT
(In reply to comment #1)
> Ready for approval, contigent on three things being fixed (listed below):
> - remove tabs
> - check with legal on licensing (looks more liberal than LGPL so I'm personally
> OK with it)
> - include debian/copyright

LLGPL is on the approved license list now.
I'll make the two other changes and add this.

Thanks!

AG
Comment 4 Anthony Green 2008-07-07 11:50:12 EDT
Here are the updated files:

Spec URL: http://spindazzle.org/Fedora/common-lisp-controller.spec
SRPM URL: http://spindazzle.org/Fedora/common-lisp-controller-6.15-2.fc9.src.rpm

Note that I also upgraded from 6.12 to 6.15.

Approved?

Thanks,

AG
Comment 5 Anthony Green 2008-07-13 10:53:12 EDT
(In reply to comment #1)
> Ready for approval, contigent on three things being fixed (listed below):
> - remove tabs
> - check with legal on licensing (looks more liberal than LGPL so I'm personally
> OK with it)
> - include debian/copyright

Michael - these were all fixed.  I have a ton of Common Lisp packages depending
on this approval.  I think we're done now, but I think I need you to flip the
bit before I request my cvs tree.  Thanks!
Comment 6 Michel Alexandre Salim 2008-08-26 20:47:08 EDT
Somehow missed the bugzilla notification, sorry.

Yes, all issues are resolved. APPROVED.
Comment 7 Anthony Green 2008-08-29 03:09:07 EDT
New Package CVS Request
=======================
Package Name: common-lisp-controller
Short Description: Common Lisp source and compiler manager
Owners: green
Branches: F-9
InitialCC:
Comment 8 Kevin Fenzi 2008-08-30 16:58:35 EDT
cvs done.
Comment 9 Peter Robinson 2008-10-11 10:51:33 EDT
Closing - in rawhide
Comment 10 Anthony Green 2010-11-12 16:53:56 EST
Package Change Request
======================
Package Name: common-lisp-controller
New Branches: el6
Owners: green
InitialCC: rdieter

I'd like to bring common-lisp-controller into el6 for sbcl and other lisps.
Comment 11 Jason Tibbitts 2010-11-13 11:48:07 EST
Git done (by process-git-requests).
Comment 12 Rex Dieter 2015-02-06 10:45:53 EST
Package Change Request
======================
Package Name: common-lisp-controller
New Branches: epel7
Owners: green
InitialCC: rdieter
Comment 13 Gwyn Ciesla 2015-02-06 12:42:49 EST
Git done (by process-git-requests).