Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 625236 - =net-misc/radvd-2.14: In file included from includes.h:106:0, from recv.c:17: /usr/include/linux/if_arp.h:114:8: error: redefinition of 'struct arpreq'
Summary: =net-misc/radvd-2.14: In file included from includes.h:106:0, from recv.c:17:...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal minor
Assignee: Matthew Thode ( prometheanfire )
URL:
Whiteboard:
Keywords: PATCH
: 644778 (view as bug list)
Depends on: 637520
Blocks:
  Show dependency tree
 
Reported: 2017-07-15 18:59 UTC by Murilo Morais
Modified: 2018-11-25 11:21 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
Compile log (build.log,8.96 KB, text/x-log)
2017-07-15 18:59 UTC, Murilo Morais 		Details
includes.h patch (00-radvd_includes.h.patch,551 bytes, patch)
2017-07-15 19:04 UTC, Murilo Morais 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Murilo Morais 2017-07-15 18:59:20 UTC 
Created attachment 484910 [details]
Compile log

The net-misc/radvd-2.14 package fails to compile with the vanilla kernel headers 4.12.1.
Comment 1 Murilo Morais 2017-07-15 19:04:46 UTC 
Created attachment 484912 [details, diff]
includes.h patch
Comment 2 Murilo Morais 2017-07-15 19:05:27 UTC 
(In reply to Murilo Morais from comment #1)
> Created attachment 484912 [details, diff] [details, diff]
> includes.h patch

I was reading the file "includes.h", there were two fields including the "/usr/include/linux/if_arp.h" header resulting in the redefinition of '...' error, I created a patch to correct and compile correctly.
Comment 3 Murilo Morais 2017-07-15 19:09:17 UTC 
(In reply to Murilo Morais from comment #2)
> (In reply to Murilo Morais from comment #1)
> > Created attachment 484912 [details, diff] [details, diff] [details, diff]
> > includes.h patch
> 
> I was reading the file "includes.h", there were two fields including the
> "/usr/include/linux/if_arp.h" header resulting in the redefinition of '...'
> error, I created a patch to correct and compile correctly.

I noticed that the headers are different, my mistake. :/
Comment 4 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2017-07-20 21:19:31 UTC 
I'd like to cc robbat on this since he actually worked on radvd upstream.
Comment 6 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2017-09-10 20:56:05 UTC 
It looks like this is included in 2.17, it sounds like that should be the next stable...
Comment 7 Austin English (RETIRED) gentoo-dev 2017-11-14 19:44:37 UTC 
(In reply to Matthew Thode ( prometheanfire ) from comment #6)
> It looks like this is included in 2.17, it sounds like that should be the
> next stable...

Yep. 2.14 fails for me, but 2.17 works without patching.
Comment 8 Brian Evans (RETIRED) gentoo-dev 2018-01-24 20:50:54 UTC 
*** Bug 644778 has been marked as a duplicate of this bug. ***
Comment 9 Larry the Git Cow gentoo-dev 2018-11-25 11:21:25 UTC 
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8fad7a82a85f3f6622425bc425c9818e8a76850c

commit 8fad7a82a85f3f6622425bc425c9818e8a76850c
Author:     Sergei Trofimovich <slyfox@gentoo.org>
AuthorDate: 2018-11-25 11:21:05 +0000
Commit:     Sergei Trofimovich <slyfox@gentoo.org>
CommitDate: 2018-11-25 11:21:05 +0000

    net-misc/radvd: fix build aganst linux-headers, bug #625236
    
    Patch by Murilo Morais
    Closes: https://bugs.gentoo.org/625236
    Package-Manager: Portage-2.3.52, Repoman-2.3.12
    Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>

 net-misc/radvd/files/radvd-2.14-headers.patch | 26 ++++++++++++++++++++++++++
 net-misc/radvd/radvd-2.14.ebuild              |  6 +++++-
 2 files changed, 31 insertions(+), 1 deletion(-)