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

Prevent makefile recursion through per-project properties or a command-line kill-switch #15

Closed
davereid opened this issue Aug 20, 2013 · 6 comments
Closed

Prevent makefile recursion through per-project properties or a command-line kill-switch #15

davereid opened this issue Aug 20, 2013 · 6 comments
Assignees
@jhedstrom

Comments

@davereid
Copy link
Contributor

Copied from https://drupal.org/node/1989174
@ghost ghost assigned jhedstrom Sep 3, 2013
@runephilosof
Copy link

The drupal issue contains a patch

@jhedstrom
Copy link
Member

Recent commits in 6.x have cleaned up recursion quite a bit (no longer will a recursive makefile overwrite a project that has been specified by a higher level makefile). Are people still thinking this is necessary?

@fago
Copy link
Contributor

fago commented Oct 10, 2013

Yes, I think that would be valuable still.

@grasmash
Copy link
Contributor

I would also find this valuable.

@grasmash
Copy link
Contributor

I've made a pull request using the patch located at https://drupal.org/node/1989174:

#320

jhedstrom added a commit that referenced this issue Dec 10, 2013
@jhedstrom
Merge pull request #320 from grasmash/1989174-do-recurse-property
bd805cf 
#15 Prevent makefile recursion through per-project properties
@jhedstrom
Copy link
Member

Thanks!

@ghost ghost mentioned this issue Jun 14, 2016
Open
@mxtyl mxtyl mentioned this issue May 10, 2021
Closed
@AppLEaDaY AppLEaDaY mentioned this issue Jan 21, 2023
Closed
@dereliahmet1 dereliahmet1 mentioned this issue Mar 23, 2023
Closed
@yotkadata yotkadata mentioned this issue Aug 2, 2023
Closed
@vcmnetwork vcmnetwork mentioned this issue Feb 12, 2024
Closed
@elaman elaman mentioned this issue Apr 8, 2024
Closed
@claudiu-cristea claudiu-cristea mentioned this issue May 6, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhedstrom jhedstrom

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@davereid @jhedstrom @fago @grasmash @runephilosof