From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] beagle: handle import of environments in files with CRLF as line endings
Date: Sun, 13 May 2012 22:18:40 +0200 [thread overview]
Message-ID: <4FB01720.90402@ahsoftware.de> (raw)
In-Reply-To: <201205132057.47247.marex@denx.de>
On 13.05.2012 20:57, Marek Vasut wrote:
> Dear Wolfgang Denk,
>
>> Dear Marek Vasut,
>>
>> In message<201205131909.49488.marek.vasut@gmail.com> you wrote:
>>>> - "env import -t $loadaddr $filesize\0" \
>>>> + "env import -t -r $loadaddr $filesize\0" \
>>>
>>> Not everyone importing env on beagle use broken tools ;-)
>>
>> It's not a problem of using broken tools - the problem is of ignorant
>> people _not_ using decade old, working tools like dos2unix if they
>> need them.
>
> tr -d '\r' won't work? :-)
>
>> As I mentionewd a coule of times before, I am seriously tempted to
>> ignore this "problem".
>
> Can't the wincrap people be taught to use cygwin? Or possibly some Windows (R)
> (C)(TM)(???) rebuilt version of tr -d '\r' ?
Sorry, it seems you just are unable or yiu don't want to understand the
problem. So here is my last message on this topic, trying to explain it
for you.
env import -t is a handy solution to give users (not only devs) a
possibility to modify variables (options) used to boot some OS. They
don't have to deal with the u-boot command-line and, depending how it's
used, env import -t is an alternative to persistent modify the env
without the need for NAND.
So one just tells the user(s) that if they want to change some option
they just have to create a text file on a specific place which contains
a line like e.g.
foo=bar
The only problem is that such a description currently only works for
Linux users. I don't know with what devices and users you are dealing,
but there are some people out in the wild which don't know (much) about
Linux, or even about the difference in line endings between text files
creates using Windows or Linux. Some of them even just want to use their
device.
And in addition, the resulting problems hare very hard to diagnose,
because there will be no obvious error message when something contains a
trailing CR.
It would be interesting how long you would need to diagnose such a
problem without knowing about that problem, but we now will never know.
Now I needed about 10 mails to describe a problem from which I thought
it's totally obvious. Sorry, but I accept that I'm totally unable to
deal with this list in any, for me reasonable, time frame.
Regards,
Alexander
next prev parent reply other threads:[~2012-05-13 20:18 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-13 12:50 [U-Boot] [PATCH 1/2] Add option -r to env import to allow import of text files with CRLF as line endings Alexander Holler
2012-05-13 12:50 ` [U-Boot] [PATCH 2/2] beagle: handle import of environments in " Alexander Holler
2012-05-13 17:09 ` Marek Vasut
2012-05-13 17:56 ` Alexander Holler
2012-05-13 18:52 ` Marek Vasut
2012-05-13 18:58 ` Alexander Holler
2012-05-13 18:49 ` Wolfgang Denk
2012-05-13 18:57 ` Marek Vasut
2012-05-13 19:26 ` Wolfgang Denk
2012-05-13 19:45 ` Marek Vasut
2012-05-13 19:53 ` Wolfgang Denk
2012-05-13 20:18 ` Alexander Holler [this message]
2012-05-13 21:38 ` Wolfgang Denk
2012-05-14 8:29 ` Alexander Holler
2012-05-14 9:07 ` Alexander Holler
2012-05-14 9:21 ` Alexander Holler
2012-05-13 21:47 ` Måns Rullgård
2012-05-14 16:02 ` Tom Rini
2012-05-14 17:04 ` Alexander Holler
2012-05-14 18:45 ` Marek Vasut
2012-05-14 22:23 ` Alexander Holler
2012-05-13 12:58 ` [U-Boot] [PATCH 1/2 v2] Add option -r to env import to allow import of text " Alexander Holler
2012-05-13 18:45 ` [U-Boot] [PATCH 1/2] " Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FB01720.90402@ahsoftware.de \
--to=holler@ahsoftware.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox