Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Factorisation of NUCLEO_F302R8 and F334R8 cmsis folders #640

Merged
merged 5 commits into from
Nov 4, 2014
Merged

Factorisation of NUCLEO_F302R8 and F334R8 cmsis folders #640

merged 5 commits into from
Nov 4, 2014

Conversation

bcostm
Copy link
Contributor

@bcostm bcostm commented Nov 3, 2014

  • STM32Cube driver moved to TARGET_STM32F3 folder

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 3, 2014

Hi,

seems the previous merge for NUCLEO_L053R8 broke the mbed library. We should look at it. I'll run the build from master and check the latest merges.

@ohagendorf
Copy link
Contributor

Yes, it is the same error message. And these two pull requests (#639 and 640) are the first after adding the L053 to build_travis.py.
The strange thing is, that both pull requests run successfully through a local build_travis.

@bcostm
Copy link
Contributor Author

bcostm commented Nov 3, 2014

I have launched this command:
build.py -m NUCLEO_F334R8 -t uARM -j 4 -c --dsp --fat
as Bogdan suggested and every thing is ok. I did also the same for F302R8 and L053R8 and also ok.
Is this what you mean by Travis test ??? If not what is the command ?

@ohagendorf
Copy link
Contributor

build_travis.py is a Python script in workspace_tools folder. It defines in a build_list variable several targets, toolchains and libraries. These are build automatically after a pull request.

The command line above is just one of these combinations. I our case the NUCLEO_L053R8 with gcc_arm build failes. The target is new in the script. This pull request and #639 are the first with it.

I had the same situation. My local builds, also the complete test with build_travis are OK.
I don't think that your pull request or my one in #639 are the problem. It has something to do with l053.

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 3, 2014

I just run it locally, it builds.. have to investigate

@bcostm
Copy link
Contributor Author

bcostm commented Nov 3, 2014

Can you read the Details of the Travis CI build in GitHub ? When I click on the Details link nothing is displayed for me. So I cannot help too much...

Update: It was due to Internet Explorer. Using Chrome I can see the log.

I know. It is due to the file name. There is a "L" instead of "l"... We need to change the line:
include "stm32l0xx_hal_rcc_ex.h"

I remember having this error in the past... and I think if we use Windows we don't have any issue. If we use Linux yes...

@bogdanm
Copy link
Contributor

bogdanm commented Nov 3, 2014

This is the same issue as the one described in pull request #639

0xc0170 added a commit that referenced this pull request Nov 4, 2014
Targets: Factorisation of NUCLEO_F302R8 and F334R8 cmsis folders
@0xc0170 0xc0170 merged commit 5a868b1 into ARMmbed:master Nov 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants