Bug 184921 - update to new version
Summary: update to new version
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: convmv
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nils Philippsen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-03-10 08:57 UTC by Patrice Dumas
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-03-10 13:36:12 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
patch with update, remove md5sum use and minor changes (1.10 KB, patch)
2006-03-10 08:57 UTC, Patrice Dumas
no flags Details | Diff

Description Patrice Dumas 2006-03-10 08:57:34 UTC
Description of problem:
There is a new version available. I attach a patch.

I also made a change, I removed the md5sum of files check, I find it not usefull
as the archive is allready checked, and other minor changes. 

Version-Release number of selected component (if applicable):

1.08-3

How reproducible:
Not relevant

Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Patrice Dumas 2006-03-10 08:57:34 UTC
Created attachment 125937 [details]
patch with update, remove md5sum use and minor changes

Comment 2 Nils Philippsen 2006-03-10 13:36:12 UTC
convmv-1.09-1 is building, but I didn't remove the MD5 check (as it might catch
upstream packaging errors) as well as I didn't change the defattr() statement
(both mean the same).

Comment 3 Patrice Dumas 2006-03-10 13:49:35 UTC
I though the defattr with - in the end was the preferred form, but I cannot find
it in the wiki, I must have been dreaming...

As for the md5sum, I doubt it is usefull, as the md5sum are regenerated by the
upstream. I removed it because it added a buildrequires and I doubt about its
usefullness, but if you are convinced it is usefull, it's indeed your choice to
keep it or not.

Comment 4 Nils Philippsen 2006-03-10 16:45:17 UTC
I think as md5sum should be installed on all system (and isn't that big), it
doesn't hurt to leave it an as a measure against upstream botching their package
build process.


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