Re: [patch] arptables: some patches from debian

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Bart. Thank you for review/applying patches. Could you also apply
attached patch to fix not respecting LDFLAGS issue (very similar to one
I've sent you long time ago for ebtables :) )

--
Peter.

Ð ÐÐÑ, 09/10/2010 Ð 21:17 +0200, Bart De Schuymer ÐÐÑÐÑ:
> Hello Peter,
> 
> Patch 2 and 3 are committed.
> For the '*' part of patch 1 I'd rather remove the -v usage in 
> arptables-restore. This will require fixing the reason why -v is used in 
> the first place. I wasn't aware this is due to a bug: the -i and -o 
> interfaces aren't printed unless -v is specified. I'll look into this.
> 
> cheers,
> Bart
> 
> On 09-10-10 15:17, Peter Volkov wrote:
> > Hi again. What is the best way to contact arptables developers? My
> > previous patch was left without attention, should I resend it? In
> > attachment there are three fixes from debian:
> >
> > 1. arptables_save patch makes arp_tables don't resolve host names and
> > don't convert '*' interface names to any. Remove '*' interface names.
> > 2. manpage patch removes old version from man page
> > 3. is a patch from Jeroen van Wolffelaar to make arptables --proto-type
> > also accept hexadecimal inputs (ethernet protocol numbers are often
> > specfied in hex, not decimal), using standard strtol() behaviour (hex
> > iff starts with 0x).

=== modified file 'Makefile'
--- Makefile	2010-09-15 11:51:49 +0000
+++ Makefile	2010-09-15 11:52:56 +0000
@@ -31,7 +31,7 @@
 	$(CC) $(CFLAGS) -c -o $@ $<
 
 arptables: arptables-standalone.o arptables.o libarptc/libarptc.o $(EXT_OBJS)
-	$(CC) $(CFLAGS) -o $@ $^
+	$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^
 
 $(DESTDIR)$(MANDIR)/man8/arptables.8: arptables.8
 	mkdir -p $(@D)


[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux