Summary: | net-analyzer/honeyd w/ dev-libs/libevent-2 - tagging.h:89: error: expected declaration specifiers or ‘...’ before ‘(’ token | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | Gentoo Netmon project <netmon> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | blueness, jer, lidel, treecleaner |
Priority: | High | Keywords: | PMASKED |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://code.google.com/p/honeyd/issues/detail?id=19 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 414997, 333077, 366081 | ||
Attachments: |
Build log
Rename honeyd's encode_int to encode__int (two underscores) |
Description
Diego Elio Pettenò (RETIRED)
![]() Created attachment 243287 [details]
Build log
Oddly enough, honeyd and libevent are from the same author. (In reply to comment #2) > Oddly enough, honeyd and libevent are from the same author. > Its precisely because its the same author that he made use of aspects of the libevent API that are not documented and were never meant to be exposed. There are two problems with this packages: 1) there is a collision between encode_int defined as a macro in libevent-2's event2/tag_compat.h and in honeyd's tagger.h/.c. That's easily fixed by renaming the function in honeyd. I'll provide the patch in my next post. 2) the harder problem is the dereferncing of struct evbuffer objects. evbuffer changed in deep ways between libevent-1 and -2. In the former it was just one element whereas in the latter it was replaced by a linked list. Upstream libevent his helping me with this. Created attachment 259592 [details, diff]
Rename honeyd's encode_int to encode__int (two underscores)
I couldn't think of a better name than encode__int with two rather than one underscore. I'll be happy to change it to something else.
honeyd now DEPENDs on =dev-libs/libevent-1*. Has upstream done anything lately? I'd like to get rid of libevent-1. Nothing as I see: http://code.google.com/p/honeyd/source/list But debian still maintains this: http://patch-tracker.debian.org/patch/debianonly/view/honeyd/1.5c-8 Probably correct workaround is to slot libevent. (In reply to comment #6) > Has upstream done anything lately? I'd like to get rid of libevent-1. When I was pushing through libevent-2 I found nothing from upstream. I then tried to recode honeyd with the new libevent API with help from Nick Mathewson, but I hit some real snags and gave up. Maybe slotting is the way to go. I don't think SLOTting is an option when only one unmaintained (by upstream) packages needs it, especially when upstream is the author of both. maybe should be treecleaned as seems upstream no longer takes care about it :/ (In reply to comment #10) > maybe should be treecleaned as seems upstream no longer takes care about it > :/ +1 I tried but it makes use of too much stuff that is now hidden in libevent-2 and should never have been exposed in libevent-1. Upstream was no help. It's gone. p.mask is gone too. |