Control: tags -1 + patch

Hi Raoul,

2016-09-17 16:31 GMT+02:00 Raoul Borenius <boren...@dfn.de>:
> Hello Balint,
>
> On Thu, Sep 15, 2016 at 11:40:57AM +0200, Balint Reczey wrote:
>>
>> I have already asked for many exceptions on d-d:
>> https://lists.debian.org/debian-devel/2016/09/msg00277.html
>>
>> If you agree with the proposed change please add i2util to the list and
>> maybe support the request. It would be nice to see at least one positive
>> answer to call it a discussion. :-)
>
> I added my package to the list but I did not vote for
> your change, sorry. I'm no C developer so I do not think that
> my vote would be backed by the needed technical know how ;-)
> Your proposal sounds valid to me though, crossing my fingers that
> the developers will agree to it.

OK, thank you.

>
> I created a new package for i2util:
>
> https://mentors.debian.net/package/i2util
>
> I included
>
>         export DEB_BUILD_MAINT_OPTIONS = hardening=+bindnow
>         DPKG_EXPORT_BUILDFLAGS = 1
>         include /usr/share/dpkg/buildflags.mk
>
> at the top of the rules file. Is that suffient in our case or do
> I need to add
>
>         export DEB_BUILD_MAINT_OPTIONS = hardening=+all
>
> ?
>
> My understanding is that the latter is not needed with the newest
> build environment.

I suggest using the following patch which is simpler and adds -fPIC:

--- i2util-1.6/debian/rules
+++ i2util-1.6/debian/rules
@@ -1,10 +1,10 @@
 #!/usr/bin/make -f

-#export DEB_BUILD_MAINT_OPTIONS = hardening=+all

-export DEB_BUILD_MAINT_OPTIONS = hardening=+bindnow
-DPKG_EXPORT_BUILDFLAGS = 1
-include /usr/share/dpkg/buildflags.mk
+export DEB_CFLAGS_MAINT_APPEND = -fPIC
+export DEB_LDFLAGS_MAINT_APPEND = -fPIC
+
+export DEB_BUILD_MAINT_OPTIONS=hardening=+all

 override_dh_auto_clean:
     [ ! -f Makefile ] || $(MAKE) distclean

I think you can also use the default dh_auto_configure, but I have not
tested that.

>
> If the package is ok as it is, would you mind uploading it to
> the archive? Im no DD so I do not have upload rights.

It is recommended to work with the same sponsors and as I see
Martin kindly sponsored your last (iperf3) uploads. If Martin can't
sponsor this upload I would happily do it but I need some more
time to review the package.

Thanks,
Balint

Reply via email to