Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 564160 - net-proxy/haproxy (9999 aka dev): DeviceAtlas support
Summary: net-proxy/haproxy (9999 aka dev): DeviceAtlas support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Christian Ruppert (idl0r)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-10-26 10:48 UTC by David Carlier
Modified: 2016-12-13 00:07 UTC (History)
1 user (show)

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


Attachments
DeviceAtlas option (haproxy-9999.ebuild.diff,1.52 KB, patch)
2015-10-26 10:48 UTC, David Carlier
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Carlier 2015-10-26 10:48:45 UTC
Created attachment 415508 [details, diff]
DeviceAtlas option

Haproxy 1.6 came with mobile device detection optional support, hence enabling it via custom USE flag. Can be applied to future 1.6.1 ebuild as well.

Hope it s useful.
Comment 1 Bertrand Jacquin 2016-10-05 21:18:39 UTC
David, having an ebuild for DeviceAtlas would be a more straight forward option for user. This package is not shipped with Gentoo portage so far. Can we work on that first ?
Comment 2 David Carlier 2016-10-05 21:26:05 UTC
For sure ! But it is due to the fact you need to register in this page https://deviceatlas.com/deviceatlas-haproxy-module 

in order to get the source code. Same "problem" applied to the FreeBSD port version I submitted awhile ago

https://svnweb.freebsd.org/ports/head/net/haproxy/Makefile?revision=421150&view=markup

Kind regards.
Comment 3 Bertrand Jacquin 2016-10-05 23:58:59 UTC
Preliminary work can be found on https://github.com/gentoo/gentoo/pull/2487

A small patch was needed in order to link to the .so file created by device-atlas-api-c instead of compiling files from /usr/include (dev-libs/device-atlas-api-c does not install the mentioned .c files but anyway that would result with sandbox violation). 

It would be great to have a proper 'install' target in device-atlas-api-c cmake files in order to make installation less hacky. May I request this for future release ?

dev-libs/device-atlas-api-c does not contain any license file so for now, license is 'public-domain'
Comment 4 David Carlier 2016-10-06 04:04:15 UTC
Good suggestion about install target will keep it in mind.

Cheers.
Comment 5 Bertrand Jacquin 2016-11-08 22:37:45 UTC
(In reply to David Carlier from comment #4)
> Good suggestion about install target will keep it in mind.

Please note that input is requested from you David in the github merge request ;)
Comment 6 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2016-12-05 18:34:10 UTC
The HAProxy parts are in place for 1.6.10 and 1.7 now (along with support for other implementations of device detection: 51Degrees & WURFL).

We still need ebuilds for the API/libraries of those device detection libraries, which should be in separate bugs.
Comment 7 Bertrand Jacquin 2016-12-13 00:07:03 UTC
Current ebuild is borken, please see https://github.com/gentoo/gentoo/pull/2487 for proper DeviceAtlas implementation. This bug should be reopened. Thanks