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

configure.ac: warn about unknown '--with-' / '--enable-' options #3029

Closed
mc-butler opened this issue Jul 3, 2013 · 7 comments
Closed

configure.ac: warn about unknown '--with-' / '--enable-' options #3029

mc-butler opened this issue Jul 3, 2013 · 7 comments
Assignees
@aborodin
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone
4.8.11

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3029
Reporter slyfox (@trofi)

As we use 'AC_CONFIG_SUBDIRS' macro for samba sources
option checking is disabled for us.

Before the patch:

$ ./configure --with-i-have-misspelled-editor-for-edit
...
# no warnings

The patch enables warnings back only for top-level ./configure:

$ ./configure --with-i-have-misspelled-editor-for-edit
...
configure: WARNING: unrecognized options: --with-i-have-misspelled-editor-for-edit

Note

Original attachments:
@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Jul 3, 2013 at 15:19 UTC

mc-4.8.9-unknown-opts.patch

@mc-butler
Copy link
Author

Changed by slyfox (@trofi) on Aug 7, 2013 at 22:11 UTC (comment 1)

  • Branch state changed from no branch to on review

branch: 3029_warn_about_unknown_switches
[1599510170c1abca5ed3a18f8f5ecc9c9b8aab20]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Sep 2, 2013 at 9:46 UTC (comment 2)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 5, 2013 at 10:43 UTC (comment 3)

  • Owner set to andrew_b
  • Status changed from new to accepted
  • Milestone changed from Future Releases to 4.8.11

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 5, 2013 at 10:43 UTC (comment 4)

  • Votes changed from slavazanko to slavazanko andrew_b
  • Branch state changed from on review to approved

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 5, 2013 at 10:47 UTC (comment 5)

  • Status changed from accepted to testing
  • Resolution set to fixed
  • Branch state changed from approved to merged
  • Votes changed from slavazanko andrew_b to committed-master

Merged to master: [d362fc9].

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 5, 2013 at 10:48 UTC (comment 6)

  • Status changed from testing to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aborodin aborodin

Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone
4.8.11
Development

No branches or pull requests
2 participants
@aborodin @mc-butler