Skip to content
This repository was archived by the owner on Nov 6, 2022. It is now read-only.

Commit 652b015

Browse files
ShaRoseindutny
authored andcommitted
makefile: fix dangling symlink in uninstall
Fix symbolic links so they are now relative, and also removes a dangling symlink that was left there with make uninstall (libhttp_parser.so was left pointing to a non-existant file previously). Don't hard-code the library name, match the file names of `make install`. PR-URL: #391 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Fedor Indutny <[email protected]>
1 parent 4cefc0f commit 652b015

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

Makefile

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,19 +133,20 @@ tags: http_parser.c http_parser.h test.c
133133
install: library
134134
$(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
135135
$(INSTALL) -D $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(LIBNAME)
136-
ln -s $(LIBDIR)/$(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
137-
ln -s $(LIBDIR)/$(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SOLIBNAME).$(SOEXT)
136+
ln -s $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
137+
ln -s $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SOLIBNAME).$(SOEXT)
138138

139139
install-strip: library
140140
$(INSTALL) -D http_parser.h $(DESTDIR)$(INCLUDEDIR)/http_parser.h
141141
$(INSTALL) -D -s $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(LIBNAME)
142-
ln -s $(LIBDIR)/$(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
143-
ln -s $(LIBDIR)/$(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SOLIBNAME).$(SOEXT)
142+
ln -s $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SONAME)
143+
ln -s $(LIBNAME) $(DESTDIR)$(LIBDIR)/$(SOLIBNAME).$(SOEXT)
144144

145145
uninstall:
146146
rm $(DESTDIR)$(INCLUDEDIR)/http_parser.h
147+
rm $(DESTDIR)$(LIBDIR)/$(SOLIBNAME).$(SOEXT)
147148
rm $(DESTDIR)$(LIBDIR)/$(SONAME)
148-
rm $(LIBDIR)/libhttp_parser.so
149+
rm $(DESTDIR)$(LIBDIR)/$(LIBNAME)
149150

150151
clean:
151152
rm -f *.o *.a tags test test_fast test_g \

0 commit comments

Comments
 (0)