From Fedora Project Wiki
mNo edit summary |
m (internal link cleaning) |
||
(4 intermediate revisions by 2 users not shown) | |||
Line 2: | Line 2: | ||
In guidelines in common errors section | In guidelines in common errors section | ||
[[Packaging/Guidelines#Rpmlint_Errors]] | |||
there is the following phrase: | there is the following phrase: | ||
<pre> | <pre> | ||
Line 16: | Line 16: | ||
or dos2unix. | or dos2unix. | ||
</pre> | </pre> | ||
Here also <code>%{__sed}</code> changed to simply <code>sed</code> to be consistent with other parts of guidelines. | Here also <code>%{__sed}</code> changed to simply <code>sed</code> to be consistent with other parts of guidelines and wiki mark-up added to separate sed command, but this is not the essential part of proposal. | ||
[[Category:Archived packaging guideline drafts]] | |||
Latest revision as of 07:49, 18 September 2016
Proposal to explicitly allow dos2unix
In guidelines in common errors section Packaging/Guidelines#Rpmlint_Errors there is the following phrase:
This error occurs because of DOS line breaks in a file. Fix it with sed in the %prep section: %{__sed} -i 's/\r//' src/somefile -- DONT use dos2unix, that can cause build fail on FC3.
Because FC3 is not supported for long time, I'm proposing to change it to:
This error occurs because of DOS line breaks in a file. Fix it in the %prep section with sed: <code>sed -i 's/\r//' src/somefile</code> or dos2unix.
Here also %{__sed}
changed to simply sed
to be consistent with other parts of guidelines and wiki mark-up added to separate sed command, but this is not the essential part of proposal.