From owner-freebsd-ports@FreeBSD.ORG Sun Sep 22 09:43:24 2013 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 80559E94 for ; Sun, 22 Sep 2013 09:43:24 +0000 (UTC) (envelope-from spankthespam@gmail.com) Received: from mail-qe0-x235.google.com (mail-qe0-x235.google.com [IPv6:2607:f8b0:400d:c02::235]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 3F65B2532 for ; Sun, 22 Sep 2013 09:43:24 +0000 (UTC) Received: by mail-qe0-f53.google.com with SMTP id jy17so1377465qeb.12 for ; Sun, 22 Sep 2013 02:43:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=oB/HeUz3Q6DLpjkSjEf0GrgAzrvoiX9A8VP5CiqeOYc=; b=gDA6txKlzT0/0rBCmftdcN5otPUUvRgC05w8kS5EPHQ6j674ctvjCgTNjtVT7nX0Qp nFZg1TnJrGVDqV6ASgb5aBgMLQv9yfiM7X/rk7ZXujzsO7WPqqkK5awF/H1N+GoumnOZ tvT1jYFQ88cECAR2E9tTvzfb4+A/yPduU8LIXzGeYzyZ/UH0TQsRF3P022bFHUjGeH7u sTc+73l+2lW1hEfX/QzD24M2Qt3pUxU0jevqbLbgY8WX4PcMAvaJIAW6T5MCvzAlihBs Z/mBOQdLZ34oZDDYwZuCAa/dFn01bGVqTZfdrDw6vAlAgE1CicopmHVwaYT2OHzYnlZg WrYg== MIME-Version: 1.0 X-Received: by 10.224.16.8 with SMTP id m8mr16723603qaa.28.1379843003313; Sun, 22 Sep 2013 02:43:23 -0700 (PDT) Received: by 10.229.205.71 with HTTP; Sun, 22 Sep 2013 02:43:23 -0700 (PDT) In-Reply-To: <523E1D31.5060302@shatow.net> References: <20130920203737.GE74496@admin.xzibition.com> <523E1D31.5060302@shatow.net> Date: Sun, 22 Sep 2013 10:43:23 +0100 Message-ID: Subject: Re: Fwd: Help in testing Basho Riak port From: Big Lebowski To: Bryan Drewery Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: freebsd-ports X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 22 Sep 2013 09:43:24 -0000 On Sat, Sep 21, 2013 at 11:26 PM, Bryan Drewery wrote: > On 9/21/2013 5:08 PM, Big Lebowski wrote: > > Hi, > > > > Thanks for your comments, see mine below. > > > > > > On Fri, Sep 20, 2013 at 9:37 PM, Bryan Drewery wrote: > > > >> On Fri, Sep 20, 2013 at 06:57:52PM +0100, Big Lebowski wrote: > >>> Hi list! > >>> > >>> I've been working for couple last days on porting Basho Riak database > >>> (latest version 1.4.2) and finally I think it is ready to be presented: > >>> https://www.dropbox.com/s/2ztu2bdiip1u2un/riak.tgz > >> > >> MASTER_SITES= > >> http://s3.amazonaws.com/downloads.basho.com/riak/1.4/1.4.2/ \ > >> > >> http://downloads.basho.com.s3.amazonaws.com/riak/1.4/1.4.2/ > >> > >> Use ${PORTVERSION} instead of 1.4.2 > >> > > > > Fixed. > > > > > >> > >> USES= ${GMAKE} > >> > >> Use 'gmake', not ${GMAKE} here. > >> > > > > Fixed. > > > > > >> > >> Fails to build on 8.3 i386: > >> > >> db/version_set.cc:59: warning: this decimal constant is unsigned only > in > >> ISO C90 > >> db/version_set.cc:59: warning: this decimal constant is unsigned only > in > >> ISO C90 > >> db/version_set.cc:60: error: integer constant is too large for 'long' > >> type > >> db/version_set.cc:60: error: integer constant is too large for 'long' > >> type > >> db/version_set.cc:61: error: integer constant is too large for 'long' > >> type > >> db/version_set.cc:61: error: integer constant is too large for 'long' > >> type > >> db/version_set.cc:62: error: integer constant is too large for 'long' > >> type > >> db/version_set.cc:62: error: integer constant is too large for 'long' > >> type > >> table/filter_block.cc: In member function 'bool > >> leveldb::FilterBlockReader::KeyMayMatch(uint64_t, const > leveldb::Slice&)': > >> table/filter_block.cc:112: warning: comparison between signed and > >> unsigned integer expressions > >> util/env_posix.cc: In constructor > >> 'leveldb::::PosixEnv::PosixEnv()': > >> util/env_posix.cc:788: warning: unused variable 'ts' > >> > >> > >> gmake[1]: *** [libleveldb.so.1.9] Error 1 > >> gmake[1]: Leaving directory > >> > `/wrkdirs/usr/ports/databases/riak/work/riak-1.4.2/deps/eleveldb/c_src/leveldb' > >> > > > > Can you provide any more details on your testing environment, like 32/64 > > bit, GCC version, any compile optimizations in make.conf and so on? > > Unfortunately, I dont have any 8.x machine to test it, so I wasnt able to > > see that before. Could you also try to build it on 8.3 using CLANG? > > 8.3 i386 (32 bit). GCC 4.2. This error is of too much data for 32bit > variable. Clang won't help. > > Here is the issue upstream as well with a recommended fix and a SO > explaining more: > > https://github.com/basho/leveldb/issues/87 > > http://stackoverflow.com/questions/5541560/integer-constant-is-too-large-for-long-type It seems like its a bug in Riak itself, and I wonder if it should be me, who's fixing it? Given the fact, the fix have not been accepted for two months now, that it is (as its author writes) still giving warnings, and given that I've zero knowledge of C, does it make sense to make this unaccepted, uncomplete and unknown code a patch for the port? If there would be Nginx error prohibiting building it on certain enviroments, would Nginx port owner be fixing it, or should it rather be a Nginx authors problem? B. > > > > > > > >> > >> > >> Instead of bsd.port.pre.mk ... bsd.port.post.mk, use > bsd.port.options.mk... > >> bsd.port.mk. Although, it doesn't look like you even need that. Just > use > >> bsd.port.mk at the end, no 2nd include. > >> > > > > I dont think I can get away with only using bsd.port.mk - I've tried > that, > > and the port fails miserably, somwhere on the level of interpretation of > > the Makefile, where it is missing many macros, so I've stayed with two > > includes. > > > > > >> > >> > >> Otherwise, good work. I will pkg-build test any updates you send out. > >> > > > > Thanks, great to hear that. Latest version will be posted in response to > my > > original message. > > > > B. > > > > > >> > >>> > >>> Please, grab the port and test it on anything you can, break it in any > >> way > >>> possible, and comment on anything you see that's been done wrong - I am > >>> open to any suggestion on how to make it worthy candidate for send-pr > >> with > >>> port submission. > >>> > >>> Any help will be highly appreciated and will motivate me to port more > >> cool > >>> software to FreeBSD! ;) > >>> > >>> Kind regards, > >>> S. > >>> _______________________________________________ > >>> freebsd-ports@freebsd.org mailing list > >>> http://lists.freebsd.org/mailman/listinfo/freebsd-ports > >>> To unsubscribe, send any mail to " > freebsd-ports-unsubscribe@freebsd.org" > >> > > _______________________________________________ > > freebsd-ports@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-ports > > To unsubscribe, send any mail to "freebsd-ports-unsubscribe@freebsd.org" > > > > >