openwall / openwall/john

Autoconf update

Open
#4,738 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C
Stars
13.6k
Forks
2.6k
PR merge metrics
No merged PRs in 30d

Description

As noted in https://github.com/openwall/john/pull/4735#pullrequestreview-670278268, the output of ./configure --help has inconsistent capitalization.

On another note: When working on #4735 I noticed latest autoconf (2.71) wanted to update our configure.ac - some will happen semi automatically on request but even after that, some macros we use are deprecated. I simply downgraded autoconf to 2.69 (same as latest Ubuntu LTS use) instead, keeping that PR on topic, but at some point in time we'll probably want/need to do a 10,000 mile service on the whole ship wreck. While at that we also have #3566...

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the capitalization inconsistencies in ./configure --help, then inspect configure.ac using the context from PR #4735. Check what autoconf 2.71 would update and which macros are deprecated, while considering the related work in #3566. Done means the configuration help is consistent and the project’s autoconf setup has been comprehensively updated.

Written by the indexing model from the issue text.

Assessment

Domain
build-system
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.