Ticket #6 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Unify the Makefile

Reported by: ccowart Owned by: blee
Priority: critical Milestone: confman-1.9b
Component: Linux Support Version: 1.7.2
Keywords: Cc:

Description

The Makefile could use some serious work.

I'd like to see a single, cross-platform Makefile. If that means gmake, then gmake it will be.

It should also be written to be more generalized.

Change History

comment:1 Changed 3 years ago by ccowart

  • Owner changed from ccowart to blee
  • Status changed from new to assigned

If you've got a Linux box to do some testing on, could you also make sure the GnuMakefile? does the right thing?

comment:2 Changed 3 years ago by blee

(In [291]) * GNU sed is 'sed' in Linux. How do we make this cross-platform?

See #6

comment:3 Changed 3 years ago by blee

(In [292]) * Replacing '.unconfig' target with a more sensible 'clean' target

that removes the .config file

See #6

comment:4 Changed 3 years ago by blee

(In [294]) * Adding 'clean' target to GNUmakefile

See #6

comment:5 Changed 3 years ago by blee

(In [295]) * Making GNUmakefile consistent with the other Makefile

See #6

comment:6 Changed 3 years ago by blee

(In [296]) * Fixing PREFIX in Makefile

See #6

comment:7 Changed 3 years ago by blee

  • Status changed from assigned to accepted

comment:8 Changed 3 years ago by blee

(In [297]) * Taking a stab at using GNU Autotools

See #6

comment:9 Changed 3 years ago by blee

(In [302]) This is the big commit to use GNU Autotools. However, Makefile.am
could use some serious cleanup.

Also:

  • abspath installed to libexec/confman
  • ABSPATH created as a defaults variable
  • confman_completions renamed to confman_completion
  • confman_completion installed to etc/bash_completion.d so it is automagically recognized by the system bash_completion

See #6

comment:10 Changed 3 years ago by blee

(In [303]) * Adding 'chmod +x' for executables after do_subst

See #6

comment:11 Changed 3 years ago by blee

(In [306]) Some Autotools cleanup:

  • /var is no longer hard-coded (instead use localstatedir)
  • confman.conf.sample is now created by autotools so that it has relevant defaults

Additionally:

  • Changing default of CONF_SSH_KEY to /root/.ssh/id_rsa

See #6

comment:12 Changed 3 years ago by blee

(In [307]) * confman.8 and confman.conf.5 generated by autotools

See #6

comment:13 Changed 3 years ago by blee

(In [309]) * Fixing CLEANFILES

See #6

comment:14 Changed 3 years ago by blee

  • Status changed from accepted to closed
  • Resolution set to fixed

Autotools is good to go. This is done.

Note: See TracTickets for help on using tickets.