Date: Sun, 29 Aug 2010 11:23:11 +0100 From: Chris Rees <utisoft@gmail.com> To: Rob Farmer <rfarmer@predatorlabs.net> Cc: cvs-ports@freebsd.org, Alexey Dokuchaev <danfe@freebsd.org>, Li-Wen Hsu <lwhsu@freebsd.org>, cvs-all@freebsd.org, ports-committers@freebsd.org Subject: Re: cvs commit: ports/java/jgraphx Makefile distinfo Message-ID: <AANLkTinqUUV4v=SGnoq_T8onxss5SkyRm259x04NaGjv@mail.gmail.com> In-Reply-To: <AANLkTik8O4fpVHU%2B_pFGTRBapTtMx%2B5fJgk4M8QnZsCw@mail.gmail.com> References: <201008221158.o7MBwce6093897@repoman.freebsd.org> <AANLkTinwzkmw3%2BkSjddODtzFSCD-5RBb1gU48%2BM6w58E@mail.gmail.com> <20100827062503.GA85184@FreeBSD.org> <AANLkTinnsfdVVRi39HEp24bXMM2ykWydnmt%2BiEdDSqHH@mail.gmail.com> <AANLkTik8O4fpVHU%2B_pFGTRBapTtMx%2B5fJgk4M8QnZsCw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--0016e6d77e7be7ccad048ef3bdc3 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 29 August 2010 01:11, Rob Farmer <rfarmer@predatorlabs.net> wrote: > On Sat, Aug 28, 2010 at 2:28 AM, Chris Rees <utisoft@gmail.com> wrote: >> 2010/8/27 Alexey Dokuchaev <danfe@freebsd.org>: >>> 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--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTinqUUV4v=SGnoq_T8onxss5SkyRm259x04NaGjv>