ARMmbed / mbed-os

Arm Mbed OS is a platform operating system designed for the internet of things
https://mbed.com
Other
4.66k stars 2.97k forks source link

ARM and uARM will conflict with build.py #15

Closed ytsuboi closed 11 years ago

ytsuboi commented 11 years ago

Issue #14 was not solved. Here is procedure.

  1. workspace_tools\build.py --clean -r -m xxxx -t ARM
  2. workspace_tools\build.py --clean -r -m xxxx -t uARM
  3. make.py -m xxxx -t uARM -p 32
  4. make.py -m xxxx -t ARM -p 32

So, you will get error message like this.

Build Options: debug-info

>>> BUILD PROJECT: BASIC (xxxx, ARM)
Link: basic
[ERROR] Error: L6267E: Input objects request both standardlib and microlib. Please choose using the --library_type option.
Finished: 0 information, 0 warning and 1 error messages.
emilmont commented 11 years ago

Hi Ytsuboi, this issue was fixed in:

If you check out the latest code from the "mbedmicro/mbed" repository this test sequence produce the desired result:

  1. workspace_tools\build.py --clean -r -m LPC11U24 -t ARM
  2. workspace_tools\build.py --clean -r -m LPC11U24 -t uARM
  3. workspace_tools\make.py -m LPC11U24 -t uARM -p 32
  4. workspace_tools\make.py -m LPC11U24 -t ARM -p 32

Before opening other tickets, could you please take the time to test the above on a clean checkout of the latest code?

Thanks, Emilio

ytsuboi commented 11 years ago

Hi Emilio-san,

Oh, I'm sorry I took a mistake when I wrote the procedure. It was 59, not 32.

Your procedure works, but this will show an error.

  1. workspace_tools\build.py --clean -r -m LPC11U24 -t ARM
  2. workspace_tools\build.py --clean -r -m LPC11U24 -t uARM
  3. workspace_tools\make.py -m LPC11U24 -t uARM -p 59
  4. workspace_tools\make.py -m LPC11U24 -t ARM -p 59

Of cource, I tried with latest code. Regards, Yoshi

emilmont commented 11 years ago

Thanks Yoshi-san, I can now reproduce this issue.

emilmont commented 11 years ago

Yes, we were not using the new build convention for the objects path when building libraries. Fixed: https://github.com/mbedmicro/mbed/commit/003c48cd44d7898b859a3bb54eb316170206b881

Thanks, Emilio