This is the mail archive of the
ecos-bugs@sourceware.org
mailing list for the eCos project.
[Bug 1000761] eCos support for MPC5xxx MCUs
- From: bugzilla-daemon at bugs dot ecos dot sourceware dot org
- To: ecos-bugs at ecos dot sourceware dot org
- Date: Mon, 1 Oct 2012 10:57:52 +0100
- Subject: [Bug 1000761] eCos support for MPC5xxx MCUs
- Auto-submitted: auto-generated
- References: <bug-1000761-13@http.bugs.ecos.sourceware.org/>
Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1000761
Bernd Edlinger <bernd.edlinger@hotmail.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bernd.edlinger@hotmail.de
--- Comment #60 from Bernd Edlinger <bernd.edlinger@hotmail.de> 2012-10-01 10:57:39 BST ---
Hello Stefan,
I was looking at this just for curiosity, and if it has a potential
to break other ports in any way.
Now there is one thing that is unclear to me, You said:
> - some small SED script lines accross multiple locations (e.g. memalloc,
> rules.mak) to work with MinGW build toolchains. I have seen many requests on
> Forums for different architectures, where people were looking for this, so this
> should be of generic interest. At this point in time the VLE enabled toolchains
> are only available in MinGW build form.
Well, I see there are changes in the sed scripts in 3 patches:
Attachment #1866, Attachment #1867 and Attachment #1868.
You changed
@sed -e '/^ *\\/d' -e "s#.*: #$@: #" $(@:.o.d=.tmp) > $@
to
@sed -e 's!c:/cygwin!!g' -e 's/c://g' -e '/^ *\\/d' -e "s#.*: #$@: #"
$(@:.o.d=.tmp) > $@
@printf "\n" >> $@
I assume you have a GCC built under MinGW and the make built under Cygwin.
and you try to remove drive letters to make the Cygwin-make understand the
dependencies.
But what I do not understand is this:
1. ":" is a quite normal character to a linux file system.
so should'nt that all be controlled by a compile option?
2. this change seems only to handle the case when cygwin is
installed under c:\cywin, however Cygwin could also be installed
in d:\cygwin, or even d:\tools\Cygwin.
3. at one place it looks slightly different:
@sed -e 's!c:/cygwin!!g' -e 's/c://g' -e 's!c:\cygwin!!g' ppc_offsets.tmp >
ppc_offsets1.tmp
should'nt this c:\ also be handled at the other places?
4. especially replacing every "c:" with "" looks wrong,
because if a DOS-file is "X:\path\file.c" its Cywin-Path
looks like "/cygdrive/x/path/file.c"
5. Are we sure that "printf" is always installed, like "sed"?
I have never seen this tool was used before, why dont you use
an "echo" here which is alredy used at many other places?
Regards,
Bernd Edlinger
--
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.