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.

Sign up for GitHub

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

Jump to bottom

Cleaning up Makefiles #161

Closed
yoogx opened this issue May 31, 2018 · 1 comment
Closed

Cleaning up Makefiles #161

yoogx opened this issue May 31, 2018 · 1 comment
Assignees
@yoogx
Labels
polyorb-hi-c TASTE

Comments

@yoogx
Copy link
Contributor

yoogx commented May 31, 2018

Several actions are required to clean-up target-specific makefiles

  • remove obsolete one
  • update macros to avoid collision with third-party makefiles
@yoogx yoogx self-assigned this May 31, 2018
yoogx added a commit to OpenAADL/polyorb-hi-c that referenced this issue May 31, 2018
@yoogx
* Remove obsolete platforms
d0a40f1 
        For OpenAADL/ocarina#161
yoogx added a commit to OpenAADL/polyorb-hi-c that referenced this issue Jun 1, 2018
@yoogx
* Change TARGET_CFLAGS to POHI_CFLAGS to avoid collision with RTOS m…
b4d826d 
…acro

	  Clean-up of RTEMS makefile to use existing macros

	For OpenAADL/ocarina#161
yoogx added a commit that referenced this issue Jun 1, 2018
@yoogx
* Remove obsolete platforms
4410108 
	For #161
yoogx added a commit that referenced this issue Jun 5, 2018
@yoogx
* Remove obsolete platform
e72bab4 
        For #161
@yoogx yoogx mentioned this issue Jun 25, 2018
Closed
@yoogx
Copy link
Contributor Author

yoogx commented Sep 5, 2018 •
edited
Loading

In addition, we should strictly adhere to GNU canonical names, see here
https://www.gnu.org/software/make/manual/make.html#Implicit-Variables

In particular, for RTEMS and others
LDFLAGS must only have path info
LDLIBS (LD_LIBS on RTEMS see share/rtems5/make/README) must have library info

yoogx added a commit to OpenAADL/polyorb-hi-c that referenced this issue Dec 20, 2018
@yoogx
* Handle special case for library handling for RTEMS
ede84b0 
        For OpenAADL/ocarina#161
@yoogx yoogx closed this as completed Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yoogx yoogx

Labels
polyorb-hi-c TASTE
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yoogx