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

common: allow appending extra CFLAGS/LDFLAGS via command line #54

Merged
merged 1 commit into from
Nov 10, 2014
Merged

common: allow appending extra CFLAGS/LDFLAGS via command line #54

merged 1 commit into from
Nov 10, 2014

Conversation

krzycz
Copy link
Contributor

@krzycz krzycz commented Nov 10, 2014

With this change, user may provide the extra flags CFLAGS/LDFLAGS using
the following syntax:

Example #1:

make EXTRA_CFLAGS=-finstrument-functions

Example #2:

CFLAGS=-finstrument-functions make

@krzycz
commmon: allow appending extra CFLAGS/LDFLAGS via command line
9b70e63 
With this change, user may provide the extra flags CFLAGS/LDFLAGS using
the following syntax:

Example #1:
> make EXTRA_CFLAGS=-finstrument-functions <target>

Example #2:
> CFLAGS=-finstrument-functions make <target>
andyrudoff pushed a commit that referenced this pull request Nov 10, 2014
Merge pull request #54 from krzycz/common-makefile_extra_cflags
e9f28b8 
common: allow appending extra CFLAGS/LDFLAGS via command line
@andyrudoff andyrudoff merged commit e9f28b8 into pmem:master Nov 10, 2014
@krzycz krzycz deleted the common-makefile_extra_cflags branch December 3, 2014 19:04
lplewa added a commit to lplewa/pmdk that referenced this pull request Feb 24, 2022
@lplewa
Squashed 'src/deps/miniasync/' changes from 93283586e..dd06ea790
555662f 
dd06ea790 Merge pull request pmem#54 from lplewa/vdm
9c6390dcb masync: make vdm and future header only
e23ab21b1 Merge pull request pmem#52 from wlemkows/KW-fixes
ad4651146 common: fix KW issues
bab51d2fc Merge pull request pmem#51 from DamianDuy/fixNonStandardExtensionWinWarnings
0c232c340 common: remove unnecessary wrappers for allocators
d2ca2230f Merge pull request pmem#49 from DamianDuy/fixTypeMismatchWinWarnings
a53362da8 common: fix type mismatch warnings on Windows

git-subtree-dir: src/deps/miniasync
git-subtree-split: dd06ea7907b27b8cdafaf957488e74dd1cb921a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krzycz @andyrudoff