All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/4] Add option -r to env import to allow import of text files with CRLF as line endings
Date: Wed, 06 Aug 2014 13:48:15 +0200	[thread overview]
Message-ID: <53E215FF.30105@ahsoftware.de> (raw)
In-Reply-To: <53E20F1F.1000306@ahsoftware.de>

Am 06.08.2014 13:18, schrieb Alexander Holler:
> Am 06.08.2014 12:44, schrieb M?ns Rullg?rd:
>> Alexander Holler <holler@ahsoftware.de> writes:
>>
>>> Am 06.08.2014 08:43, schrieb Wolfgang Denk:
>>>> Dear Alexander,
>>>>
>>>> In message <53DFDC99.2020206@ahsoftware.de> you wrote:
>>>>>
>>>>> At least I've understood it such that nobody still has an objection=20
>>>>> against the new feature for "env import -t" (-r).
>>>>
>>>> I object if it would be added without maintaining symmetry with "env
>>>> export".
>>>
>>> As explained I don't know how to add symmetry. It's impossible to
>>> export text concurrently in both formats.
>>
>> What is the problem with ignoring \r on input and not writing it on
>> output?
>>
>
> None. Please read the mails before.

And just in case it has become difficult to follow this thread because 
the subject should have changed to something else, I think what Wolfgang 
Denk wants is an option to extent "env export" to export the environment 
to text files with CRLF.

I've made a suggestion here:

http://lists.denx.de/pipermail/u-boot/2014-August/185272.html

But as there was no response until now (I'm not impatient, I'm just 
mentioning it again because I think you've missed about what Wolfgang 
Denk started to discuss). it's just an assumption from me, I don't have 
any clue what he means with symmetry in regard to that "-r".

And without any consensus I will not spend the time to write the few 
lines of source to implement that (I don't need it). As written before, 
I don't even care much if -r for "env export -t" ends up in mainline U-Boot.

So there is no problem, Wolfgang Denk and I are just discussing if and 
how to extend "env export". That is at least how I do now understand 
this thread about the simple patch I've posted some years ago.

Regards,

Alexander Holler

  reply	other threads:[~2014-08-06 11:48 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-14 15:49 [U-Boot] [PATCH 0/4] Add option -r to env import to allow import of text files with CRLF as line endings Alexander Holler
2014-07-14 15:49 ` [U-Boot] [PATCH 1/4] " Alexander Holler
2014-07-22 19:23   ` [U-Boot] [U-Boot, " Tom Rini
2014-07-30 22:47   ` [U-Boot] [PATCH " Stephen Warren
2014-07-31 19:51     ` Tom Rini
2014-07-31 19:57       ` Stephen Warren
2014-08-14  8:25         ` Alexander Holler
2014-08-14 15:49           ` Stephen Warren
2014-08-14 18:41             ` Robert Nelson
2014-08-14 19:38               ` Tom Rini
2014-08-14 19:50               ` Stephen Warren
2014-08-14 20:53                 ` Tom Rini
2014-08-14 21:05                   ` Alexander Holler
2014-08-14 21:35                   ` Stephen Warren
2014-08-14 21:44                     ` Alexander Holler
2014-08-14 19:38             ` Alexander Holler
2014-08-14 19:51               ` Stephen Warren
2014-08-14 19:59                 ` Alexander Holler
2014-08-14 20:08                   ` Stephen Warren
2014-08-14 20:39                     ` Alexander Holler
2014-08-01 12:08   ` Wolfgang Denk
2014-08-02 21:09     ` Alexander Holler
2014-08-03 16:38       ` Alexander Holler
2014-08-03 17:51         ` Wolfgang Denk
2014-08-04  6:47           ` Alexander Holler
2014-08-04  7:00             ` Alexander Holler
2014-08-04 10:00             ` Måns Rullgård
2014-08-04 19:18               ` Alexander Holler
2014-08-06  6:43                 ` Wolfgang Denk
2014-08-06 10:02                   ` Alexander Holler
2014-08-06 10:28                     ` Alexander Holler
2014-08-06 10:44                     ` Måns Rullgård
2014-08-06 11:18                       ` Alexander Holler
2014-08-06 11:48                         ` Alexander Holler [this message]
2014-08-03 17:47       ` Wolfgang Denk
2014-07-14 15:49 ` [U-Boot] [PATCH 2/4] omap3_beagle: handle import of environments in " Alexander Holler
2014-07-22 19:23   ` [U-Boot] [U-Boot, " Tom Rini
2014-07-14 15:49 ` [U-Boot] [PATCH 3/4] am335x_evm: " Alexander Holler
2014-07-22 19:23   ` [U-Boot] [U-Boot, " Tom Rini
2014-07-14 15:49 ` [U-Boot] [PATCH 4/4] rpi_b: " Alexander Holler
2014-07-22 19:23   ` [U-Boot] [U-Boot, " Tom Rini

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=53E215FF.30105@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.