Fix script removal (#310)
The `script` variable you were referring to previously is actually the relative path to a script, for example `bin/mw`. This fails to remove the script, because, obviously, the path is incorrect. I'd suggest renaming `script` to `scriptPath` in line 13 and `shared` to `sharedPath` in line 18 (or whatever variable names you prefer that make it clearer) to avoid confusion in the future. Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
This commit is contained in:
		
				
					committed by
					
						
						Luke Smith
					
				
			
			
				
	
			
			
			
						parent
						
							c81a1bd525
						
					
				
				
					commit
					4904fcf51f
				
			
							
								
								
									
										5
									
								
								Makefile
									
									
									
									
									
								
							
							
						
						
									
										5
									
								
								Makefile
									
									
									
									
									
								
							@@ -26,8 +26,9 @@ install:
 | 
				
			|||||||
	cp -f mw.1 $(DESTDIR)$(MANPREFIX)/man1/mw.1
 | 
						cp -f mw.1 $(DESTDIR)$(MANPREFIX)/man1/mw.1
 | 
				
			||||||
 | 
					
 | 
				
			||||||
uninstall:
 | 
					uninstall:
 | 
				
			||||||
	for script in bin/*; do \
 | 
						for scriptPath in bin/*; do \
 | 
				
			||||||
		rm -f $(DESTDIR)$(PREFIX)/bin/$$script; \
 | 
							scriptFile=$$(basename $$scriptPath); \
 | 
				
			||||||
 | 
							rm -f $(DESTDIR)$(PREFIX)/bin/$$scriptFile; \
 | 
				
			||||||
	done
 | 
						done
 | 
				
			||||||
	rm -rf $(DESTDIR)$(PREFIX)/share/mutt-wizard
 | 
						rm -rf $(DESTDIR)$(PREFIX)/share/mutt-wizard
 | 
				
			||||||
	rm -f $(DESTDIR)$(MANPREFIX)/man1/mw.1
 | 
						rm -f $(DESTDIR)$(MANPREFIX)/man1/mw.1
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user