Skip to content

Commit

Permalink
added a Makefile demo for test_aco_synopsis.c
Browse files Browse the repository at this point in the history
  • Loading branch information
hnes committed Jul 12, 2018
1 parent c193525 commit 7290791
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
TARGET = test_aco_synopsis
CC = gcc
CFLAGS = -g -O2 -Wall -Werror

test_aco_synopsis : acosw.o aco.o test_aco_synopsis.o
cc -o test_aco_synopsis acosw.o aco.o test_aco_synopsis.o

acosw.o : acosw.S
$(CC) $(CFLAGS) -c acosw.S -o acosw.o
aco.o : aco.c aco.h aco_assert_override.h
$(CC) $(CFLAGS) -c aco.c -o aco.o
test_aco_synopsis.o : test_aco_synopsis.c aco.h aco_assert_override.h
$(CC) $(CFLAGS) -c test_aco_synopsis.c -o test_aco_synopsis.o

clean :
rm acosw.o aco.o test_aco_synopsis.o test_aco_synopsis

7 comments on commit 7290791

@elfring
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you add such a make script when better variants can be generated by other known tools?

@hnes
Copy link
Owner Author

@hnes hnes commented on 7290791 Jul 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This Makefile is just a temporary demonstration and will be substituted when our formal build system generator arrives (I should have commented on it clearly).

@elfring
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Such a make script might be temporarily useful for your own software build experiments while advanced development approaches will be safer and more powerful.
You could separate further development ideas on corresponding topic branches, couldn't you?

@hnes
Copy link
Owner Author

@hnes hnes commented on 7290791 Jul 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just moved it into the dev branch.
Thank you very much for your advice ;)

@elfring
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hnes
Copy link
Owner Author

@hnes hnes commented on 7290791 Jul 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's very possible.

@elfring
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest then to give desired branches clearer names.

Please sign in to comment.