Re: Thunderbird rust sed

"Rainer Fiebig" ([email protected] via blfs-dev Mailing List) <[email protected]>
Newsgroups gmane.linux.lfs.beyond.devel
Message-ID <[email protected]>
Am 14.08.25 um 16:52 schrieb Xi Ruoyao ([email protected] via blfs-dev
Mailing List):
> On Thu, 2025-08-14 at 14:57 +0200, Rainer Fiebig wrote:
>> Am 14.08.25 um 02:00 schrieb Joe Locash ([email protected] via blfs-
>> dev
>> Mailing List):
>>> I'm not sure why the sed for the rust fix needs to convert CRLF's
>>> but if
>>> you want to keep the same consistency in the file, this sed is
>>> easier:
>>>   sed -i '1i#![warn(dangerous_implicit_autorefs)]\r'
>>> third_party/rust/allocator-api2/src/stable/vec/mod.rs
>>> and you can loose the footnote at the same time ;)
>> I like your approach.  But I don't think that it would obviate the
>> footnote (which I appreciate) as the "\r" at the end still needs to be
>> explained.
>>
>> With "i.orig" I'd also give the less sed-gifted but curious user a
>> chance to understand what's been done. So my suggestion for a simpler
>> and more user-friendly sed would be:
>>
>> sed -e '1i #![warn(dangerous_implicit_autorefs)]\r'\
>>     -i.orig third_party/rust/allocator-api2/src/stable/vec/mod.rs
> 
> I'd like to know first if the \r is really needed...
Without it emacs and vim display a ^M at the end of every line except
the added one.  Checked that.  It's also explained in the footnote.

Rainer

-- 
http://lists.linuxfromscratch.org/sympa/info/blfs-dev
Unsubscribe: See the above information page
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.