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.

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

Makefile: fix warning "jobserver unavailable: using -j1" #2489

Merged
merged 1 commit into from Sep 11, 2020

Conversation

madmartin
Copy link
Contributor

@madmartin madmartin commented Sep 10, 2020

Pull Request Details:
  • Type: Bug Fix

  • Impact: Normal

  • Reference to related issue (URL):
    N/A

  • How was this pull request tested?
    The issue occurred when creating a gentoo ebuild for the 2.6 release

  • Brief description of the changes in this pull request:
    when running "make install" make throws warnings
    make[1]: warning: jobserver unavailable: using -j1. Add '+' to parent make rule.
    Fix: do not call "make" inside a Makefile, use "$(MAKE)" instead
    For reference, see
    https://www.gnu.org/software/make/manual/html_node/MAKE-Variable.html

when running "make install" make throws warnings
  make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
Fix: do not call "make" inside a Makefile, use "$(MAKE)" instead
@gdha gdha self-assigned this Sep 10, 2020
@gdha gdha added the minor bug An alternative or workaround exists label Sep 10, 2020
@gdha gdha requested review from jsmeix and gdha September 10, 2020 07:25
Copy link
Member

@gdha gdha left a comment

Choose a reason for hiding this comment

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

@madmartin good catch - thank you for the enhancement of the Makefile

Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

I know nothing about make details but according to
https://www.gnu.org/software/make/manual/html_node/MAKE-Variable.html

Recursive make commands should always use the variable MAKE,
not the explicit command name ‘make’

the change in this pull request seems to be the right thing to do in general
so I approve it.

@jsmeix jsmeix added this to the ReaR v2.7 milestone Sep 10, 2020
@gdha gdha merged commit be5a7bb into rear:master Sep 11, 2020
@madmartin madmartin deleted the fix-makefile branch September 12, 2020 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed / solved / done minor bug An alternative or workaround exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants