Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 460814 - net-analyzer/rrdtool: add support for LuaJIT
Summary: net-analyzer/rrdtool: add support for LuaJIT
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Netmon project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2013-03-08 18:59 UTC by Vadim A. Misbakh-Soloviov (mva)
Modified: 2022-04-05 17:44 UTC (History)
1 user (show)

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


Attachments
rrdtool-1.4.7-r3.ebuild (rrdtool-1.4.7-r2.ebuild,4.32 KB, text/plain)
2013-03-08 19:00 UTC, Vadim A. Misbakh-Soloviov (mva)
Details
new ebuild (based on rrdtool-1.4.8-r1) (rrdtool-1.4.8-r1.ebuild,4.52 KB, text/plain)
2014-06-08 12:17 UTC, Vadim A. Misbakh-Soloviov (mva)
Details
Patch to =net-analyzer/rrdtool-1.4.8-r1 (rrdtool-luajit.patch,1.68 KB, patch)
2014-06-08 12:18 UTC, Vadim A. Misbakh-Soloviov (mva)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vadim A. Misbakh-Soloviov (mva) gentoo-dev 2013-03-08 18:59:46 UTC
Will such ebuild be okay to be commited to the tree?

Reproducible: Always
Comment 1 Vadim A. Misbakh-Soloviov (mva) gentoo-dev 2013-03-08 19:00:45 UTC
Created attachment 341332 [details]
rrdtool-1.4.7-r3.ebuild
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2013-03-09 17:15:28 UTC
Comment on attachment 341332 [details]
rrdtool-1.4.7-r3.ebuild

--- rrdtool-1.4.7-r2.ebuild     2013-03-06 17:28:54.000000000 +0100
+++ -   2013-03-09 18:15:16.313770880 +0100
@@ -1,11 +1,12 @@
 # Copyright 1999-2013 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/net-analyzer/rrdtool/rrdtool-1.4.7-r2.ebuild,v 1.3 2013/03/06 16:28:54 jer Exp $
+# $Header: $
 
 EAPI="5"
 
 GENTOO_DEPEND_ON_PERL="no"
 PYTHON_COMPAT=( python2_7 )
+
 inherit eutils distutils-r1 flag-o-matic multilib perl-module autotools
 
 DESCRIPTION="A system to store and display time-series data"
@@ -15,7 +16,7 @@
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="~alpha ~amd64 ~arm ~hppa ~ia64 ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86 ~x86-fbsd ~amd64-linux ~ia64-linux ~x86-linux ~x86-macos ~x86-solaris"
-IUSE="dbi doc +graph lua perl python ruby rrdcgi static-libs tcl tcpd"
+IUSE="dbi doc +graph lua luajit perl python ruby rrdcgi static-libs tcl tcpd"
 
 RDEPEND="
        >=dev-libs/glib-2.28.7[static-libs=]
@@ -24,9 +25,10 @@
        graph? (
                >=media-libs/libpng-1.5.10[static-libs=]
                >=x11-libs/cairo-1.10.2[svg,static-libs=]
-               >=x11-libs/pango-1.28
+       >=x11-libs/pango-1.28
        )
-       lua? ( dev-lang/lua[deprecated] )
+       lua? ( || ( virtual/lua >=dev-lang/lua-5[deprecated] dev-lang/luajit:2 ) )
+       luajit? ( dev-lang/luajit:2 )
        perl? ( dev-lang/perl )
        python? ( ${PYTHON_DEPS} )
        ruby? ( >=dev-lang/ruby-1.8.6_p287-r13 )
@@ -103,7 +105,9 @@
 }
 
 src_compile() {
-       default
+       local lua=lua;
+       use luajit && lua=luajit;
+       emake LUA_INSTALL_CMOD="$($(tc-getPKG_CONFIG) --variable INSTALL_CMOD ${lua})" LUA_INSTALL_LMOD="$($(tc-getPKG_CONFIG) --variable INSTALL_LMOD ${lua})" || die "make install failed"

        use python && distutils-r1_src_compile
 }
Comment 3 Vadim A. Misbakh-Soloviov (mva) gentoo-dev 2014-06-08 12:17:44 UTC
Created attachment 378462 [details]
new ebuild (based on rrdtool-1.4.8-r1)
Comment 4 Vadim A. Misbakh-Soloviov (mva) gentoo-dev 2014-06-08 12:18:53 UTC
Created attachment 378464 [details, diff]
Patch to =net-analyzer/rrdtool-1.4.8-r1

Alternatively, here is a patch for current in-tree ebuild
Comment 5 Vadim A. Misbakh-Soloviov (mva) gentoo-dev 2016-10-15 09:05:29 UTC
ping?
Comment 6 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2020-12-22 04:34:45 UTC
(In reply to Vadim A. Misbakh-Soloviov (mva) from comment #5)
> ping?

Just go for it (with the new eclasses, I guess).
Comment 7 Conrad Kostecki gentoo-dev 2022-04-05 17:44:52 UTC
luajit is in current version 1.7.2 and it's stable, so closing here.