Bug 1050953 - Review Request: octave-parallel - Parallel execution package for cluster computers for Octave
Summary: Review Request: octave-parallel - Parallel execution package for cluster comp...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Orion Poplawski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1050900
TreeView+ depends on / blocked
 
Reported: 2014-01-09 12:22 UTC by Thomas Sailer
Modified: 2014-01-19 04:04 UTC (History)
2 users (show)

Fixed In Version: octave-parallel-2.1.1-3.fc20
Clone Of:
Environment:
Last Closed: 2014-01-19 04:03:11 UTC
Type: Bug
Embargoed:
orion: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Thomas Sailer 2014-01-09 12:22:39 UTC
Spec URL: http://sailer.fedorapeople.org/octave-parallel.spec
SRPM URL: http://sailer.fedorapeople.org/octave-parallel-2.1.1-1.fc20.src.rpm 
Description: Parallel execution package for cluster computers for Octave

scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6378405

rpmlint octave-parallel.spec /home/sailer/rpmbuild/SRPMS/octave-parallel-2.1.1-1.fc20.src.rpm /home/sailer/rpmbuild/RPMS/x86_64/octave-parallel-2.1.1-1.fc20.x86_64.rpm /home/sailer/rpmbuild/RPMS/x86_64/octave-parallel-debuginfo-2.1.1-1.fc20.x86_64.rpm
octave-parallel.x86_64: W: obsolete-not-provided octave-forge
octave-parallel.x86_64: W: dangerous-command-in-%preun rm
octave-parallel-debuginfo.x86_64: E: non-standard-dir-perm /usr/lib/debug 0775L
octave-parallel-debuginfo.x86_64: E: non-standard-dir-perm /usr/src/debug/parallel-2.1.1 0775L
3 packages and 1 specfiles checked; 2 errors, 2 warnings.

These errors and warnings cannot be influenced in this .spec file, so they can be ignored.

Comment 1 Orion Poplawski 2014-01-09 15:36:17 UTC
- Let's drop the octave-forge Obsoletes - that was last shipped in F15/EL5.
- Drop rm -rf %{buildroot} from %install
- There is no .svnignore file
- Why are you removing the doc directory?
- No need for %defattr()

Comment 2 Thomas Sailer 2014-01-09 17:04:37 UTC
Thank you Orion for taking the review!

Here's the update:
http://sailer.fedorapeople.org/octave-parallel.spec
http://sailer.fedorapeople.org/octave-parallel-2.1.1-2.fc20.src.rpm

(In reply to Orion Poplawski from comment #1)
> - Why are you removing the doc directory?

Because the same files are added using %doc later, thus end up in /usr/share/doc/octave-parallel-*.

I can change this and leave the README* files in the original place, if you want.

Comment 3 Orion Poplawski 2014-01-09 17:28:29 UTC
Okay, removing doc seems okay in this case.  Be careful though in case docs are added in the future.

Final item - I don't see the GFDL license anywhere.

Comment 4 Thomas Sailer 2014-01-09 17:33:23 UTC
Urgh, that's a copy&paste leftover, so I removed GFDL.

http://sailer.fedorapeople.org/octave-parallel.spec
http://sailer.fedorapeople.org/octave-parallel-2.1.1-3.fc20.src.rpm

Comment 5 Orion Poplawski 2014-01-09 17:45:57 UTC
Great, approved.

Comment 6 Thomas Sailer 2014-01-09 17:47:57 UTC
Thank you!

New Package SCM Request
=======================
Package Name: octave-parallel
Short Description: Parallel execution package for cluster computers for Octave
Owners: sailer
Branches: f19 f20
InitialCC:

Comment 7 Gwyn Ciesla 2014-01-09 18:14:11 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2014-01-09 22:54:08 UTC
octave-parallel-2.1.1-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/octave-parallel-2.1.1-3.fc19

Comment 9 Fedora Update System 2014-01-09 22:54:17 UTC
octave-parallel-2.1.1-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/octave-parallel-2.1.1-3.fc20

Comment 10 Fedora Update System 2014-01-11 08:43:09 UTC
octave-parallel-2.1.1-3.fc19 has been pushed to the Fedora 19 testing repository.

Comment 11 Fedora Update System 2014-01-19 04:03:11 UTC
octave-parallel-2.1.1-3.fc19 has been pushed to the Fedora 19 stable repository.

Comment 12 Fedora Update System 2014-01-19 04:04:05 UTC
octave-parallel-2.1.1-3.fc20 has been pushed to the Fedora 20 stable repository.


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