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

configure.ac: add support for AM_SILENT_RULES #2134

Closed
mc-butler opened this issue Apr 6, 2010 · 10 comments
Closed

configure.ac: add support for AM_SILENT_RULES #2134

mc-butler opened this issue Apr 6, 2010 · 10 comments
Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around ver: 4.7.1 Reproducible in version 4.7.1
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2134
Reporter slyfox (@trofi)
Mentions zaytsev (@zyv)

It's an automake-1.11 feature.
Thorough description of pros and cons can be found here:
http://www.flameeyes.eu/autotools-mythbuster/automake/silent.html
In short:

test-hellow % make V=0
make  all-am
make[1]: Entering directory `/home/flame/test-hellow'
  CC     hellow.o
  CCLD   hellow
make[1]: Leaving directory `/home/flame/test-hellow'

I propose to make V=0 as a default

---
Merged as [bccf557]

git log f593976..bccf557
@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Apr 6, 2010 at 17:34 UTC (comment 1)

  • Summary changed from configure.as: add support for AM_SILENT_RULES to configure.ac: add support for AM_SILENT_RULES

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Apr 6, 2010 at 17:40 UTC (comment 2)

  • Severity changed from no branch to on review
  • Owner set to slyfox
  • Status changed from new to accepted

pushed as branch:2134_am_silent_rules (commit:9dbbb664301bed3b3d1fce7f5da752bd92fbaad0)

Don't forget to remove your auto* cache!
Just running ./autogen.sh DOES NOT HELP

$ git clean -dfx

is the simplest way.

Enjoy!

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Apr 6, 2010 at 17:42 UTC (comment 3)

Hopefully clickable: [9dbbb664301bed3b3d1fce7f5da752bd92fbaad0]

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 6, 2010 at 17:53 UTC (comment 4)

  • Description edited
  • Votes set to andrew_b

This patch doesn't break the MC compilation with Automake < 1.11.

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 7, 2010 at 8:03 UTC (comment 5)

Watch the grammar/spelling:

[1457ea69f89bf81bc7bd6e59a3431c6be221a17e]

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 7, 2010 at 8:03 UTC (comment 6)

  • Cc set to zaytsev

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Apr 10, 2010 at 9:09 UTC (comment 7)

  • Severity changed from on review to approved
  • Votes changed from andrew_b to andrew_b angel_il

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Apr 10, 2010 at 9:47 UTC (comment 8)

  • Severity changed from approved to merged
  • Description edited
  • Status changed from accepted to testing
  • Resolution set to fixed
  • Votes andrew_b angel_il deleted

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Apr 10, 2010 at 9:53 UTC (comment 9)

  • Status changed from testing to closed

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 10, 2010 at 10:04 UTC (comment 10)

  • Votes set to commited-master

Merged to master: [bccf557]

git log f593976..bccf557

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: low Minor problem or easily worked around ver: 4.7.1 Reproducible in version 4.7.1
Development

No branches or pull requests

1 participant