From owner-cvs-ports@FreeBSD.ORG Sun Aug 29 10:23:43 2010 Return-Path: Delivered-To: cvs-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CE6271065679; Sun, 29 Aug 2010 10:23:43 +0000 (UTC) (envelope-from utisoft@gmail.com) Received: from mail-bw0-f54.google.com (mail-bw0-f54.google.com [209.85.214.54]) by mx1.freebsd.org (Postfix) with ESMTP id 9BAF98FC0A; Sun, 29 Aug 2010 10:23:42 +0000 (UTC) Received: by bwz20 with SMTP id 20so3613919bwz.13 for ; Sun, 29 Aug 2010 03:23:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:received:reply-to :in-reply-to:references:from:date:message-id:subject:to:cc :content-type; bh=FhPMvqOlFU+BAZ1VR6BiwHAJKrdvzvseCYSLgkK+axw=; b=BWtIUJ+K+r/NN0ybSjMsxfm3UwPx3Ea7Qp8BDn33TORftf+rgMXRV0EqrwM9GdQXkp a0yVcI7s2LMpb5bS2ZMq3syJsYby0NAnj81IoECOa9oseUc7Xr95+AxGoo0v5fhm9bi+ Xe3fQxCVHpM+mcqaFcrBJ+Imbowk+Vz0peSLI= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:reply-to:in-reply-to:references:from:date:message-id :subject:to:cc:content-type; b=d1xFWhe746ukbXq+K4tg+MRkqidHg8ekj7h0WqqgPApXaxtZRINnA84YLBGiBI9K/L cHRNU3doryxs3FTg58uGkKDTdJa6sP3swBWY7JOogtsNvg/5jSA56h7oJ4IoHYZig1Ty 05x1oqf5WWLHqYgdWUXKNxhNJ5joQfdoac3Xg= Received: by 10.204.47.38 with SMTP id l38mr2132179bkf.154.1283077421124; Sun, 29 Aug 2010 03:23:41 -0700 (PDT) MIME-Version: 1.0 Received: by 10.204.123.78 with HTTP; Sun, 29 Aug 2010 03:23:11 -0700 (PDT) In-Reply-To: References: <201008221158.o7MBwce6093897@repoman.freebsd.org> <20100827062503.GA85184@FreeBSD.org> From: Chris Rees Date: Sun, 29 Aug 2010 11:23:11 +0100 Message-ID: To: Rob Farmer Content-Type: multipart/mixed; boundary=0016e6d77e7be7ccad048ef3bdc3 Cc: cvs-ports@freebsd.org, Alexey Dokuchaev , Li-Wen Hsu , cvs-all@freebsd.org, ports-committers@freebsd.org Subject: Re: cvs commit: ports/java/jgraphx Makefile distinfo X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: utisoft@gmail.com List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 29 Aug 2010 10:23:43 -0000 --0016e6d77e7be7ccad048ef3bdc3 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 29 August 2010 01:11, Rob Farmer wrote: > On Sat, Aug 28, 2010 at 2:28 AM, Chris Rees wrote: >> 2010/8/27 Alexey Dokuchaev : >>> On Thu, Aug 26, 2010 at 09:53:32PM -0700, Rob Farmer wrote: >>>> This breaks math/scilab (which is the only dependency in the ports >>>> tree). Unfortunately, the author of jgraphx seems to completely >>>> disregard backwards compatibility and changes the API in virtually >>>> every release. >>>> >>>> I tried to patch Scilab based on their git repository (which has >>>> support for 1.4.0.1), but hundreds of revisions have passed and they >>>> have rearranged their tree a bit and added/removed some files, so it >>>> didn't go well. >>>> >>>> IMHO, we need to either create a separate jgraphx-scilab port or keep >>>> this in sync with Scilab (this is what Debian, Ubuntu, and Gentoo are >>>> doing). >>> >>> Considering Scilab is the only consumer of jgraphx, it seems having >>> special port would be an overkill. =A0I think we should keep the two in >>> sync, and probably work with upstream maintainers of both projects to >>> improve compatibility and API inheritance in the future. =A0Separate po= rt >>> of jgraphx-scilab is palliative solution, i.e. it simply increases the >>> entropy, not solving the underlying problem. >>> >>> ./danfe >> >> >> Since Scilab is the only consumer of jgraphx, I don't mind reverting it. >> >> Actually, I wrote that while trying to repair Scilab myself, so if you >> want maintainership of jgraphx too, that's fine. > > I don't want to feel like I'm stealing your ports, but I do think it > would be a good idea to have them maintained together. > >> >> Alternatively you could have it as another distfile in Scilab rather >> than depending on the port.... > > I hadn't thought about this, but it may actually be the best solution > - as far as I'm aware, the reason for having libraries in separate > ports is to allow multiple applications to use the same copy, but > given the instability of the jgraphx API, I think it is unlikely that > multiple ports could depend on one common version of jgraphx (at least > without significant patching), so the benefits of having a jgraphx > port are probably limited. Thoughts? > You're not stealing my ports, you'd be taking a logical step to fixing the madness! You are welcome to jgraphx if you want it, though having thought about the instability, maybe the alternative distfile could be a good solution. Have a look at the attached patch; the only thing i haven't done is told configure it's not a problem that jgraphx.jar isn't there yet... Of course, jgraphx is still a useful port for people who want to use it themselves in their own programming; there are plenty of other ports that aren't depended on by anything else! Chris --0016e6d77e7be7ccad048ef3bdc3 Content-Type: application/octet-stream; name="scilab.diff" Content-Disposition: attachment; filename="scilab.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gdfr34z00 ZGlmZiAtcnVOIHNjaWxhYi5vcmlnL01ha2VmaWxlIHNjaWxhYi9NYWtlZmlsZQotLS0gc2NpbGFi Lm9yaWcvTWFrZWZpbGUJMjAxMC0wNS0zMSAwMjo1OTo0OC4wMDAwMDAwMDAgKzAxMDAKKysrIHNj aWxhYi9NYWtlZmlsZQkyMDEwLTA4LTI5IDExOjA3OjQ5LjAwMDAwMDAwMCArMDEwMApAQCAtOSwx MiArOSwxOCBAQAogUE9SVFZFUlNJT049CTUuMi4yCiBQT1JUUkVWSVNJT049CTIKIENBVEVHT1JJ RVM9CW1hdGggY2FkIHBhcmFsbGVsIGphdmEKLU1BU1RFUl9TSVRFUz0JaHR0cDovL3d3dy5zY2ls YWIub3JnL2Rvd25sb2FkLyR7UE9SVFZFUlNJT059LwotRVhUUkFDVF9TVUZYPQktc3JjLnRhci5n egorTUFTVEVSX1NJVEVTPQlodHRwOi8vd3d3LnNjaWxhYi5vcmcvZG93bmxvYWQvJHtQT1JUVkVS U0lPTn0vOnNjaWxhYiBcCisJCWh0dHA6Ly93d3cuYmF5b2ZydW0ubmV0L2Rpc3QvamdyYXBoeC86 amdyYXBoeCBcCisJCWh0dHA6Ly9jZXJiZXJ1cy5iYXlvZnJ1bS5uZXQvZGlzdC9qZ3JhcGh4Lzpq Z3JhcGh4CitESVNURklMRVM9CSR7RElTVE5BTUV9LXNyYy50YXIuZ3o6c2NpbGFiIFwKKwkJJHtK R1JBUEhYX0pBUn06amdyYXBoeAogCiBNQUlOVEFJTkVSPQlyZmFybWVyQHByZWRhdG9ybGFicy5u ZXQKIENPTU1FTlQ9CVNjaWVudGlmaWMgc29mdHdhcmUgcGFja2FnZSBmb3IgbnVtZXJpY2FsIGNv bXB1dGF0aW9ucwogCitKR1JBUEhYX1ZFUlNJT049CTEuMy4wLjAKK0pHUkFQSFhfSkFSPQlqZ3Jh cGh4LSR7SkdSQVBIWF9WRVJTSU9OOkMvXC4vXy9nfS5qYXIKKwogIyBBIHBhcnRpYWwgZGVwZW5k ZW5jaWVzIGxpc3QgY2FuIGJlIGZvdW5kIGF0OgogIyBodHRwOi8vd2lraS5zY2lsYWIub3JnL0Rl cGVuZGVuY2llc19vZl9TY2lsYWJfNS5YCiBMSUJfREVQRU5EUz0JeG1sMi41OiR7UE9SVFNESVJ9 L3RleHRwcm9jL2xpYnhtbDIgXApAQCAtMjA5LDYgKzIxNSw3IEBACiAJJHtSRUlOUExBQ0VfQ01E fSAtZSAnc3wkJGphcmRpci9iYXRpay5qYXJ8JHtMT0NBTEJBU0V9L3NoYXJlL2phdmEvYmF0aWsv YmF0aWsuamFyfGcnICR7V1JLU1JDfS9jb25maWd1cmUKIAkke1JFSU5QTEFDRV9DTUR9IC1lICdz fCQkamFyZGlyL2ZvcC5qYXJ8JHtMT0NBTEJBU0V9L3NoYXJlL2phdmEvZm9wL2J1aWxkL2ZvcC5q YXJ8JyAke1dSS1NSQ30vY29uZmlndXJlCiAJJHtSRUlOUExBQ0VfQ01EfSAtZSAnc3wkJGphcmRp ci9qZXVjbGlkLWNvcmUqLmphcnwke0xPQ0FMQkFTRX0vc2hhcmUvamF2YS9qZXVjbGlkL3JlcG8v amV1Y2xpZC1jb3JlKi5qYXJ8JyAke1dSS1NSQ30vY29uZmlndXJlCisJJHtSRUlOUExBQ0VfQ01E fSAtZSAnc3wkJGphcmRpci9qZ3JhcGh4Lmphcnwke0RBVEFESVJ9L21vZHVsZXMvamdyYXBoeC5q YXJ8JyAke1dSS1NSQ30vY29uZmlndXJlCiAJIyBCdWlsZCBmaXhlcwogCSR7UkVJTlBMQUNFX0NN RH0gLWUgJ3N8JCQoQ0ZMQUdTOi1PJT0pfC1JJHtMT0NBTEJBU0V9L2luY2x1ZGUgLUwke0xPQ0FM QkFTRX0vbGlifCcgJHtXUktTUkN9L21vZHVsZXMvZWxlbWVudGFyeV9mdW5jdGlvbnMvTWFrZWZp bGUuaW4KIAkke1JFSU5QTEFDRV9DTUR9IC1lICdzfGZyZWVic2R8X19GcmVlQlNEX198ZycgJHtX UktTUkN9L21vZHVsZXMvc3BhcnNlL2luY2x1ZGVzL3NwQ29uZmlnLmgKQEAgLTIzMyw1ICsyNDAs NiBAQAogCUAke0ZJTkR9ICR7V1JLU1JDfSAtbmFtZSBcKi5vcmlnIC1kZWxldGUKIAlAJHtGSU5E fSAke1dSS1NSQ30gLW5hbWUgXCouYmFrIC1kZWxldGUKIAlAJHtGSU5EfSAke1dSS1NSQ30gLW5h bWUgLmN2c2lnbm9yZSAtZGVsZXRlCisJQCR7TUtESVJ9ICR7REFUQURJUn0vbW9kdWxlcyAmJiAk e0lOU1RBTExfREFUQX0gJHtESVNURElSfS8ke0pHUkFQSFhfSkFSfSAke0RBVEFESVJ9L21vZHVs ZXMKIAogLmluY2x1ZGUgPGJzZC5wb3J0Lm1rPgpkaWZmIC1ydU4gc2NpbGFiLm9yaWcvZGlzdGlu Zm8gc2NpbGFiL2Rpc3RpbmZvCi0tLSBzY2lsYWIub3JpZy9kaXN0aW5mbwkyMDEwLTA0LTMwIDA5 OjU0OjM5LjAwMDAwMDAwMCArMDEwMAorKysgc2NpbGFiL2Rpc3RpbmZvCTIwMTAtMDgtMjkgMTE6 MTE6MzcuMDAwMDAwMDAwICswMTAwCkBAIC0xLDMgKzEsNiBAQAogTUQ1IChzY2lsYWItNS4yLjIt c3JjLnRhci5neikgPSAzNWU4YTIyZWRiMzQ4ZjQwNTM2MGQzYTA3ZWQ0ZmFhYgogU0hBMjU2IChz Y2lsYWItNS4yLjItc3JjLnRhci5neikgPSA2Zjg4MDBiYzhhYzkxMjgxODdkMWY5YTk4YTVlNDU5 YmJmYTJiNjY5NjdjYWVkOGFiNjU5ZmYyYmE1MWYxY2YyCiBTSVpFIChzY2lsYWItNS4yLjItc3Jj LnRhci5neikgPSA1NDA4ODU4MAorTUQ1IChqZ3JhcGh4LTFfM18wXzAuamFyKSA9IGI5ZTI0ZjBi NDI4YzY3OTY5N2U3NjNlOWIwYmE1NjcxCitTSEEyNTYgKGpncmFwaHgtMV8zXzBfMC5qYXIpID0g MGYzNzI4YjQ3YzlmZDM5NzdhMDYzMjEyZjY4ZTQzNjM5YzIyODNlYjg0ZmJjZTQzNzI3NWI2Nzll NmU0YTZiMgorU0laRSAoamdyYXBoeC0xXzNfMF8wLmphcikgPSAzMjY1OTYKZGlmZiAtcnVOIHNj aWxhYi5vcmlnL3BrZy1wbGlzdCBzY2lsYWIvcGtnLXBsaXN0Ci0tLSBzY2lsYWIub3JpZy9wa2ct cGxpc3QJMjAxMC0wNC0zMCAwOTo1NDozOS4wMDAwMDAwMDAgKzAxMDAKKysrIHNjaWxhYi9wa2ct cGxpc3QJMjAxMC0wOC0yOSAxMToyMDoxMi4wMDAwMDAwMDAgKzAxMDAKQEAgLTU3MDQsNiArNTcw NCw3IEBACiAlJUdVSSUlJSVEQVRBRElSJSUvbW9kdWxlcy9qYXZhc2NpL3Rlc3RzL3VuaXRfdGVz dHMvamF2YWMuc2NpCiAlJUdVSSUlJSVEQVRBRElSJSUvbW9kdWxlcy9qYXZhc2NpL3Rlc3RzL3Vu aXRfdGVzdHMvdGVzdF9qYXZhLnNjaQogJSVHVUklJSUlREFUQURJUiUlL21vZHVsZXMvamF2YXNj aS92ZXJzaW9uLnhtbAorJSVEQVRBRElSJSUvbW9kdWxlcy9qZ3JhcGh4LmphcgogJSVEQVRBRElS JSUvbW9kdWxlcy9qdm0vY2hhbmdlbG9nLnR4dAogJSVEQVRBRElSJSUvbW9kdWxlcy9qdm0vZXRj L2p2bS5xdWl0CiAlJURBVEFESVIlJS9tb2R1bGVzL2p2bS9ldGMvanZtLnN0YXJ0Cg== --0016e6d77e7be7ccad048ef3bdc3--