Skip to content

[EXPORT][CoIDE] links to include files are added to project file #508

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

Merged
merged 2 commits into from
Sep 27, 2014

Conversation

ohagendorf
Copy link
Contributor

also for the target KL05Z, KL25Z, ARCH_PRO and LPC1768 the include files are linked into project files

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 25, 2014

One question, why source folder is named "sourcen" ?

@ohagendorf
Copy link
Contributor Author

Perhaps better names are 'Header Files' and 'Source Files'? I just had a look at Visual Studio and these names are used there.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 26, 2014

Uvision currently has src virtual folder for sources for example. You can create two virtual groups there. Headers and Sources.

Ideally, there will be groups defined in the mbed according to the folders - CMSIS, HAL , etc and all templates will support it.

changed the names of virtual directories to 'sources' and 'headers'
@ohagendorf
Copy link
Contributor Author

I've changed the virtual folder names to 'sources' and 'headers'.

The original structure with groups like CMSIS, HAL, etc. is broken up earlier when build.py is called and the files are copied to build/mbed.

It would be nice to have the original structure in the project files but the necessary change would influence everything in the build and export process.

0xc0170 added a commit that referenced this pull request Sep 27, 2014
[Export] CoIDE -  include files are in the project
@0xc0170 0xc0170 merged commit 54101fc into ARMmbed:master Sep 27, 2014
@ohagendorf ohagendorf deleted the develop_coide branch September 27, 2014 13:37
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.

2 participants