public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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