Red Hat Bugzilla – Bug 825822
Don't add unnecessary dependency on dos2unix
Last modified: 2018-04-11 09:18:30 EDT
Created attachment 587247 [details]
suggested patch on .spec file
Description of problem:
Using dos2unix in .spec file is unnecessary and it adds useless dependency on another external package.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1.Try to rebuild the package on a system with minimal installation
missing dependency on dos2unix package
we should prefer already available tools
Just because this happens to work now, doesn't mean it always will. Worst case the build requires does nothing. I wouldn't want to remove it without some way of knowing that it is guaranteed to be pulled in or that it was documented that it can be considered available. Particularly since this a build time requirement, not a run time requirement, this doesn't have much affect for most people.
(In reply to comment #1)
> Just because this happens to work now, doesn't mean it always will.
You expect sudden disappearance of sed from Linux? That would be quite interesting. Anyway ... as you wish, you are the maintainer.
This seems to be a different suggestion. Rather than drop the dependency, switch to using something that would be pulled in always. It won't be high priority, but I'll look at what we have documented as the way to strip carriage returns. If that there isn't anything, I'll look to see if there is a tool that can do that that can assumed to be present that can do that and propose documenting that in the packaging guidelines.
This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.
(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)
More information and reason for this action is here:
There is an example of using sed to do this on the packaging tricks page. sed is guaranteed to be in the buildroot. I'll change both ois and meshmagick to use sed. If I run across it in other packages I maintain (when just used for line endings, encoding issues might justify its use) I'll change it as well.
Example: sed -i "s|\r||g" file.txt
Thank you for suggesting this change. The latest rawhide build no longer buildrequires dos2unix.