On Sat, Dec 31, 2011 at 9:09 PM, Dave Reisner <[email protected]> wrote: > On Sat, Dec 31, 2011 at 09:07:12PM -0600, Dan McGee wrote: >> Signed-off-by: Dan McGee <[email protected]> >> --- >> >> If anyone has any exotic builds that this breaks in, let me know, but since >> we >> now install everything else we should probably do these too. This vastly >> simplifies the pacman-contrib PKGBUILD to say the least. >> >> contrib/Makefile.am | 6 ++++++ >> 1 files changed, 6 insertions(+), 0 deletions(-) >> >> diff --git a/contrib/Makefile.am b/contrib/Makefile.am >> index 8751fd9..73494e8 100644 >> --- a/contrib/Makefile.am >> +++ b/contrib/Makefile.am >> @@ -64,6 +64,12 @@ $(OURFILES): Makefile >> >> all-am: $(OURSCRIPTS) $(OURFILES) >> >> +install-data-local: >> + $(MKDIR_P) $(DESTDIR)$(sysconfdir)/bash_completion.d/ >> + $(INSTALL_DATA) bash_completion >> $(DESTDIR)$(sysconfdir)/bash_completion.d/pacman >> + $(MKDIR_P) $(DESTDIR)$(datarootdir)/zsh/site-functions/ >> + $(INSTALL_DATA) zsh_completion >> $(DESTDIR)$(datarootdir)/zsh/site-functions/_pacman >> + > > Don't we want an uninstall rule to match this? I recall having to add > them when I worked on the symlink voodoo for repo-add/remove and the > manpages.
Done, good call.
