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

Solve that / vs \ mystery in the objs rule... #31

Closed
xparq opened this issue Mar 27, 2023 · 0 comments
Closed

Solve that / vs \ mystery in the objs rule... #31

xparq opened this issue Mar 27, 2023 · 0 comments
Labels
question Further information is requested

Comments

@xparq
Copy link
Owner

xparq commented Mar 27, 2023 •
edited 

objs: $(src_dir)/$(units_pattern).$(SRC_EXT_)
# Amazingly, using backslashes (instead of /) in the `patsubst` below would kill the inf. rule matching! :-o
# (Despite also changing it to backslash above, in the rule, of course.
# And despite inf. rules being quite robust against / vs \ otherwise.
# What am I missing here, on this late night hour?...)
    @$(MAKE_CMD) RECURSED_FOR_COMPILING=1 DIR=$(DIR) $(custom_build_options)\
	    $(patsubst $(src_dir)/%,$(obj_dir)/%, $(subst .$(SRC_EXT_),.obj,$**))

Well, OK, the need for escaping \ was what I had been missing! :)

objs: $(src_dir)\$(units_pattern).$(SRC_EXT_)
    @$(MAKE_CMD) RECURSED_FOR_COMPILING=1 DIR=$(DIR) $(custom_build_options)\
	    $(patsubst $(src_dir)\\%,$(obj_dir)\\%, $(subst .$(SRC_EXT_),.obj,$**))
@xparq xparq added the question Further information is requested label Mar 27, 2023
@xparq xparq changed the title Solve that / vs '\' mystery in the objs rule... Solve that / vs \ mystery in the objs rule... Mar 28, 2023
xparq added a commit that referenced this issue Mar 28, 2023
@xparq
Cfg. and robustness improvements (closing 8 issues)
af3c803 
Issue #35: Capitalize the cfg. options
Issue #12: Allow customizing the main targets to build (& clean)
Issue #33: Custom build options: propagate them through recursion
Issue #10: Guard against empty base dirs in $(some_dir)/... constructs!
Issue #34: Add some tests
Issue #5: Reinforced support for spaces in the build dir itself
Issue #30: What to do with "cmdline macros can't be modified"?
Issue #31: That / vs \ mystery in the objs rule
@xparq xparq closed this as completed Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xparq