public inbox for crossgcc@sourceware.org
 help / color / mirror / Atom feed
* Fwd: Patch submission
       [not found] ` <CAJGKYO7cZ9Esh=uK0MhD62qSzq1OdjgjG8AiFLkHhoz9bdFNdw@mail.gmail.com>
@ 2013-10-04 15:30   ` Roberto A. Foglietta
  2013-10-29  0:21     ` Cody P Schafer
  0 siblings, 1 reply; 3+ messages in thread
From: Roberto A. Foglietta @ 2013-10-04 15:30 UTC (permalink / raw)
  To: crossgcc

[-- Attachment #1: Type: text/plain, Size: 324 bytes --]

Hi to all,

 I would focus your attention on

CT_LIBC_GLIBC_CONFIGPARMS="--long-option-but-short-would-fail-as-well"

 in attachment a related patch upon the last stable version 1.19.0
which fix the related printf -- missing.

 Best regards,
--
Roberto A. Foglietta
mobile: (+39) 349.33.30.697
http://roberto.foglietta.name

[-- Attachment #2: ct-ng-p1.patch --]
[-- Type: application/octet-stream, Size: 506 bytes --]

--- ./scripts/build/libc/glibc-eglibc.sh-common.orig	2013-10-04 17:12:56.091104287 +0200
+++ ./scripts/build/libc/glibc-eglibc.sh-common	2013-10-04 17:13:12.979188030 +0200
@@ -304,7 +304,7 @@ do_libc_backend_once() {
     fi
 
     # Pre-seed the configparms file with values from the config option
-    printf "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
+    printf -- "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
 
     cross_cc=$(CT_Which "${CT_TARGET}-gcc")
     extra_cc_args+=" ${extra_flags}"

[-- Attachment #3: Type: text/plain, Size: 71 bytes --]

--
For unsubscribe information see http://sourceware.org/lists.html#faq

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fwd: Patch submission
  2013-10-04 15:30   ` Fwd: Patch submission Roberto A. Foglietta
@ 2013-10-29  0:21     ` Cody P Schafer
  2013-10-29  8:44       ` Roberto A. Foglietta
  0 siblings, 1 reply; 3+ messages in thread
From: Cody P Schafer @ 2013-10-29  0:21 UTC (permalink / raw)
  To: crossgcc

On 10/04/2013 08:29 AM, Roberto A. Foglietta wrote:
> -    printf "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
> +    printf -- "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
>

This isn't portable. Is there a reason not to just do:

+    printf "%s\n" "${CT_LIBC_GLIBC_CONFIGPARMS}" > configparms

?

--
For unsubscribe information see http://sourceware.org/lists.html#faq

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fwd: Patch submission
  2013-10-29  0:21     ` Cody P Schafer
@ 2013-10-29  8:44       ` Roberto A. Foglietta
  0 siblings, 0 replies; 3+ messages in thread
From: Roberto A. Foglietta @ 2013-10-29  8:44 UTC (permalink / raw)
  To: Cody P Schafer; +Cc: crossgcc

2013/10/29 Cody P Schafer <devel-lists@codyps.com>:
> On 10/04/2013 08:29 AM, Roberto A. Foglietta wrote:
>>
>> -    printf "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
>> +    printf -- "${CT_LIBC_GLIBC_CONFIGPARMS}\n" > configparms
>>
>
> This isn't portable. Is there a reason not to just do:
>
> +    printf "%s\n" "${CT_LIBC_GLIBC_CONFIGPARMS}" > configparms
>
> ?
>

Hi Cody,

 yes it works, you are right.

example code:

a="--ciao";
printf "%s\n" "$a" && echo OK;
printf "$a" 2>/dev/null || echo ERROR

code output:

--ciao
OK
ERROR

Thanks,
-- 
Roberto A. Foglietta
mobile: (+39) 349.33.30.697
http://roberto.foglietta.name

--
For unsubscribe information see http://sourceware.org/lists.html#faq

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-10-29  8:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CAJGKYO7OouY7giQ2-X6UOwByar3ja3_fkUwx3CagJt-1Y3Lxqw@mail.gmail.com>
     [not found] ` <CAJGKYO7cZ9Esh=uK0MhD62qSzq1OdjgjG8AiFLkHhoz9bdFNdw@mail.gmail.com>
2013-10-04 15:30   ` Fwd: Patch submission Roberto A. Foglietta
2013-10-29  0:21     ` Cody P Schafer
2013-10-29  8:44       ` Roberto A. Foglietta

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).