Makefile change: uninstall target

Notices and updates
Locked
rickyboy
Posts: 607
Joined: Fri Apr 08, 2005 7:13 pm
Location: Front Royal, Virginia

Makefile change: uninstall target

Post by rickyboy »

Hello Lutz,

This would be a small change to 'Makefile' in the distro, but I recommend to change the 'uninstall' section according to this 'diff' output:

Code: Select all

204,208c204,208
<       -rm  /usr/bin/newlisp
<       -rm  /usr/bin/newlisp-tk
<       -rm  -rf /usr/share/newlisp
<       -rm  /usr/share/man/man1/newlisp.1
<       -rm  /usr/share/man/man1/newlisp-tk.1
---
>       -rm  $(bindir)/newlisp
>       -rm  $(bindir)/newlisp-tk
>       -rm  -rf $(datadir)/newlisp
>       -rm  $(mandir)/man1/newlisp.1
>       -rm  $(mandir)/man1/newlisp-tk.1
Cheers, --Ricky
(λx. x x) (λx. x x)

Lutz
Posts: 5289
Joined: Thu Sep 26, 2002 4:45 pm
Location: Pasadena, California
Contact:

Post by Lutz »

Thanks for the tip, look for it in 8.5.4

Lutz

rickyboy
Posts: 607
Joined: Fri Apr 08, 2005 7:13 pm
Location: Front Royal, Virginia

Post by rickyboy »

Ooops! I just got 8.5.4 and the Makefile is still missing a change on one line in the 'uninstall' rule. Namely, change

Code: Select all

        -rm  -rf /usr/share/newlisp
to

Code: Select all

        -rm  -rf $(datadir)/newlisp
It will then be commensurate with the consistent use of '$(datadir)/newlisp/...' in the 'install' rule.

Cheers, --Ricky
(λx. x x) (λx. x x)

Locked