Date: Sun, 29 Aug 2010 20:10:34 -0700 From: Rob Farmer <rfarmer@predatorlabs.net> To: utisoft@gmail.com, bf1783@gmail.com, wen heping <wenheping@gmail.com> 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: <AANLkTi=hBLL0076RCsOmHq14cU-F5OZ_sjybOcrT-Kho@mail.gmail.com> In-Reply-To: <AANLkTinqUUV4v=SGnoq_T8onxss5SkyRm259x04NaGjv@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> <AANLkTinqUUV4v=SGnoq_T8onxss5SkyRm259x04NaGjv@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--0016e64717acd1635d048f01ce4e Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On Sun, Aug 29, 2010 at 3:23 AM, Chris Rees <utisoft@gmail.com> wrote: > 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 i= n >>>> sync, and probably work with upstream maintainers of both projects to >>>> improve compatibility and API inheritance in the future. =A0Separate p= ort >>>> 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 > It turns out that make extract unzips the jar file, so I fixed this and moved jgraphx inside the GUI option. I also added in the fix for PR 149659 to avoid having two PORTREVISION bumps in a short time. Plus, this became MAKE_JOBS_SAFE somewhere along the line. --=20 Rob Farmer --0016e64717acd1635d048f01ce4e Content-Type: application/octet-stream; name="scilab-5.2.2_3.diff" Content-Disposition: attachment; filename="scilab-5.2.2_3.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_gdgr4ju50 SW5kZXg6IE1ha2VmaWxlCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT0KUkNTIGZpbGU6IC9ob21lL25jdnMvcG9ydHMvbWF0 aC9zY2lsYWIvTWFrZWZpbGUsdgpyZXRyaWV2aW5nIHJldmlzaW9uIDEuNjgKZGlmZiAtdSAtcjEu NjggTWFrZWZpbGUKLS0tIE1ha2VmaWxlCTMxIE1heSAyMDEwIDAxOjU5OjQ4IC0wMDAwCTEuNjgK KysrIE1ha2VmaWxlCTMwIEF1ZyAyMDEwIDAyOjUyOjIxIC0wMDAwCkBAIC03LDEwICs3LDEzIEBA CiAKIFBPUlROQU1FPQlzY2lsYWIKIFBPUlRWRVJTSU9OPQk1LjIuMgotUE9SVFJFVklTSU9OPQky CitQT1JUUkVWSVNJT049CTMKIENBVEVHT1JJRVM9CW1hdGggY2FkIHBhcmFsbGVsIGphdmEKLU1B U1RFUl9TSVRFUz0JaHR0cDovL3d3dy5zY2lsYWIub3JnL2Rvd25sb2FkLyR7UE9SVFZFUlNJT059 LwotRVhUUkFDVF9TVUZYPQktc3JjLnRhci5negorTUFTVEVSX1NJVEVTPQlodHRwOi8vd3d3LnNj aWxhYi5vcmcvZG93bmxvYWQvJHtQT1JUVkVSU0lPTn0vOnNjaWxhYiBcCisJCWh0dHA6Ly93d3cu YmF5b2ZydW0ubmV0L2Rpc3QvamdyYXBoeC86amdyYXBoeCBcCisJCWh0dHA6Ly9jZXJiZXJ1cy5i YXlvZnJ1bS5uZXQvZGlzdC9qZ3JhcGh4LzpqZ3JhcGh4CitESVNURklMRVM9CSR7RElTVE5BTUV9 LXNyYy50YXIuZ3o6c2NpbGFiCitFWFRSQUNUX09OTFk9CSR7RElTVE5BTUV9LXNyYy50YXIuZ3oK IAogTUFJTlRBSU5FUj0JcmZhcm1lckBwcmVkYXRvcmxhYnMubmV0CiBDT01NRU5UPQlTY2llbnRp ZmljIHNvZnR3YXJlIHBhY2thZ2UgZm9yIG51bWVyaWNhbCBjb21wdXRhdGlvbnMKQEAgLTIxLDcg KzI0LDcgQEAKIAkJcGNyZS4wOiR7UE9SVFNESVJ9L2RldmVsL3BjcmUgXAogCQlyZWFkbGluZS42 OiR7UE9SVFNESVJ9L2RldmVsL3JlYWRsaW5lCiAKLU1BS0VfSk9CU19VTlNBRkU9CXllcworTUFL RV9KT0JTX1NBRkU9CXllcwogCiBVU0VfRk9SVFJBTj0JeWVzCiBVU0VfR0VUVEVYVD0JeWVzCkBA IC04NCwxMyArODcsMTYgQEAKIFBMSVNUX1NVQis9CU5PR1VJPSIiCiBOT1BPUlRET0NTPQlZRVMK IC5lbHNlCitESVNURklMRVMrPQkke0pHUkFQSFhfSkFSfTpqZ3JhcGh4CitKR1JBUEhYX1ZFUlNJ T049CTEuMy4wLjAKK0pHUkFQSFhfSkFSPQlqZ3JhcGh4LSR7SkdSQVBIWF9WRVJTSU9OOkMvXC4v Xy9nfS5qYXIKKwogTElCX0RFUEVORFMrPQloZGY1LjY6JHtQT1JUU0RJUn0vc2NpZW5jZS9oZGY1 LTE4CiAKIEJVSUxEX0RFUEVORFMrPQlhbnQ6JHtQT1JUU0RJUn0vZGV2ZWwvYXBhY2hlLWFudCBc CiAJCSR7SkFWQUxJQkRJUn0vZmxleGRvY2stMC41LjEuamFyOiR7UE9SVFNESVJ9L2RldmVsL2Zs ZXhkb2NrIFwKIAkJJHtKQVZBTElCRElSfS9qb2dsLmphcjoke1BPUlRTRElSfS9ncmFwaGljcy9q b2dsIFwKIAkJJHtKQVZBTElCRElSfS9jb21tb25zLWxvZ2dpbmcuamFyOiR7UE9SVFNESVJ9L2ph dmEvamFrYXJ0YS1jb21tb25zLWxvZ2dpbmcgXAotCQkke0pBVkFMSUJESVJ9L2pncmFwaHguamFy OiR7UE9SVFNESVJ9L2phdmEvamdyYXBoeCBcCiAJCSR7SkFWQUxJQkRJUn0vbG9va3MuamFyOiR7 UE9SVFNESVJ9L2RldmVsL2xvb2tzIFwKIAkJJHtKQVZBTElCRElSfS9za2lubGYuamFyOiR7UE9S VFNESVJ9L3gxMS10b29sa2l0cy9za2lubGYgXAogCQkke0pBVkFMSUJESVJ9L2pyb3NldHRhLUFQ SS5qYXI6JHtQT1JUU0RJUn0vamF2YS9qcm9zZXR0YSBcCkBAIC0xMTIsNyArMTE4LDYgQEAKIFJV Tl9ERVBFTkRTKz0JJHtKQVZBTElCRElSfS9mbGV4ZG9jay0wLjUuMS5qYXI6JHtQT1JUU0RJUn0v ZGV2ZWwvZmxleGRvY2sgXAogCQkke0pBVkFMSUJESVJ9L2pvZ2wuamFyOiR7UE9SVFNESVJ9L2dy YXBoaWNzL2pvZ2wgXAogCQkke0pBVkFMSUJESVJ9L2NvbW1vbnMtbG9nZ2luZy5qYXI6JHtQT1JU U0RJUn0vamF2YS9qYWthcnRhLWNvbW1vbnMtbG9nZ2luZyBcCi0JCSR7SkFWQUxJQkRJUn0vamdy YXBoeC5qYXI6JHtQT1JUU0RJUn0vamF2YS9qZ3JhcGh4IFwKIAkJJHtKQVZBTElCRElSfS9sb29r cy5qYXI6JHtQT1JUU0RJUn0vZGV2ZWwvbG9va3MgXAogCQkke0pBVkFMSUJESVJ9L3NraW5sZi5q YXI6JHtQT1JUU0RJUn0veDExLXRvb2xraXRzL3NraW5sZiBcCiAJCSR7SkFWQUxJQkRJUn0vanJv c2V0dGEtQVBJLmphcjoke1BPUlRTRElSfS9qYXZhL2pyb3NldHRhIFwKQEAgLTIwOSw2ICsyMTQs OSBAQAogCSR7UkVJTlBMQUNFX0NNRH0gLWUgJ3N8JCRqYXJkaXIvYmF0aWsuamFyfCR7TE9DQUxC QVNFfS9zaGFyZS9qYXZhL2JhdGlrL2JhdGlrLmphcnxnJyAke1dSS1NSQ30vY29uZmlndXJlCiAJ JHtSRUlOUExBQ0VfQ01EfSAtZSAnc3wkJGphcmRpci9mb3AuamFyfCR7TE9DQUxCQVNFfS9zaGFy ZS9qYXZhL2ZvcC9idWlsZC9mb3AuamFyfCcgJHtXUktTUkN9L2NvbmZpZ3VyZQogCSR7UkVJTlBM QUNFX0NNRH0gLWUgJ3N8JCRqYXJkaXIvamV1Y2xpZC1jb3JlKi5qYXJ8JHtMT0NBTEJBU0V9L3No YXJlL2phdmEvamV1Y2xpZC9yZXBvL2pldWNsaWQtY29yZSouamFyfCcgJHtXUktTUkN9L2NvbmZp Z3VyZQorCSR7TUtESVJ9ICR7V1JLU1JDfS90aGlyZHBhcnR5CisJJHtDUH0gJHtESVNURElSfS8k e0pHUkFQSFhfSkFSfSAke1dSS1NSQ30vdGhpcmRwYXJ0eS9qZ3JhcGh4LmphcgorCSR7UkVJTlBM QUNFX0NNRH0gLWUgJ3N8QEpHUkFQSFhAfCR7REFUQURJUn0vbW9kdWxlcy9qZ3JhcGh4Lmphcnxn JyAke1dSS1NSQ30vZXRjL2NsYXNzcGF0aC54bWwuaW4KIAkjIEJ1aWxkIGZpeGVzCiAJJHtSRUlO UExBQ0VfQ01EfSAtZSAnc3wkJChDRkxBR1M6LU8lPSl8LUkke0xPQ0FMQkFTRX0vaW5jbHVkZSAt TCR7TE9DQUxCQVNFfS9saWJ8JyAke1dSS1NSQ30vbW9kdWxlcy9lbGVtZW50YXJ5X2Z1bmN0aW9u cy9NYWtlZmlsZS5pbgogCSR7UkVJTlBMQUNFX0NNRH0gLWUgJ3N8ZnJlZWJzZHxfX0ZyZWVCU0Rf X3xnJyAke1dSS1NSQ30vbW9kdWxlcy9zcGFyc2UvaW5jbHVkZXMvc3BDb25maWcuaApAQCAtMjIz LDYgKzIzMSw5IEBACiAJCSR7V1JLU1JDfS9tb2R1bGVzL2NhbGxfc2NpbGFiL2V4YW1wbGVzL2Nh bGxfc2NpbGFiL05FVC9WQi5ORVQvTXlfUHJvamVjdC8KIAkke1JFSU5QTEFDRV9DTUR9IC1lICdz fE15IFByb2plY3R8TXlfUHJvamVjdHxnJyAke1dSS1NSQ30vbW9kdWxlcy9jYWxsX3NjaWxhYi9l eGFtcGxlcy9jYWxsX3NjaWxhYi9ORVQvVkIuTkVUL1ZCLk5FVC52YnByb2oKIAkke1JFSU5QTEFD RV9DTUR9IC1lICdzfCQkKGxpYmRpcikvcGtnY29uZmlnfCR7UFJFRklYfS9saWJkYXRhL3BrZ2Nv bmZpZ3wnICR7V1JLU1JDfS9NYWtlZmlsZS5pbgorCSMgUnVuIGZpeGVzCisJJHtSRUlOUExBQ0Vf Q01EfSAtZSAnc3wlJUpBVkFMSUJESVIlJXwke0pBVkFMSUJESVJ9fCcgJHtXUktTUkN9L2V0Yy9s aWJyYXJ5cGF0aC54bWwKKwkke1JFSU5QTEFDRV9DTUR9IC1lICdzfCUlUFJFRklYJSV8JHtQUkVG SVh9fCcgJHtXUktTUkN9L2V0Yy9saWJyYXJ5cGF0aC54bWwKIC5pZiBkZWZpbmVkKE5PUE9SVERP Q1MpCiAJQCR7UkVJTlBMQUNFX0NNRH0gLWUgJy8lJVBPUlRET0NTJSUvZCcgJHtXUktTUkN9L01h a2VmaWxlLmluCiAuZWxzZQpAQCAtMjMzLDUgKzI0NCw2IEBACiAJQCR7RklORH0gJHtXUktTUkN9 IC1uYW1lIFwqLm9yaWcgLWRlbGV0ZQogCUAke0ZJTkR9ICR7V1JLU1JDfSAtbmFtZSBcKi5iYWsg LWRlbGV0ZQogCUAke0ZJTkR9ICR7V1JLU1JDfSAtbmFtZSAuY3ZzaWdub3JlIC1kZWxldGUKKwlA JHtNS0RJUn0gJHtEQVRBRElSfS9tb2R1bGVzICYmICR7SU5TVEFMTF9EQVRBfSAke0RJU1RESVJ9 LyR7SkdSQVBIWF9KQVJ9ICR7REFUQURJUn0vbW9kdWxlcy9qZ3JhcGh4LmphcgogCiAuaW5jbHVk ZSA8YnNkLnBvcnQubWs+CkluZGV4OiBkaXN0aW5mbwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09ClJDUyBmaWxlOiAvaG9t ZS9uY3ZzL3BvcnRzL21hdGgvc2NpbGFiL2Rpc3RpbmZvLHYKcmV0cmlldmluZyByZXZpc2lvbiAx LjE3CmRpZmYgLXUgLXIxLjE3IGRpc3RpbmZvCi0tLSBkaXN0aW5mbwkzMCBBcHIgMjAxMCAwODo1 NDozOSAtMDAwMAkxLjE3CisrKyBkaXN0aW5mbwkzMCBBdWcgMjAxMCAwMDo0NjoxOSAtMDAwMApA QCAtMSwzICsxLDYgQEAKIE1ENSAoc2NpbGFiLTUuMi4yLXNyYy50YXIuZ3opID0gMzVlOGEyMmVk YjM0OGY0MDUzNjBkM2EwN2VkNGZhYWIKIFNIQTI1NiAoc2NpbGFiLTUuMi4yLXNyYy50YXIuZ3op ID0gNmY4ODAwYmM4YWM5MTI4MTg3ZDFmOWE5OGE1ZTQ1OWJiZmEyYjY2OTY3Y2FlZDhhYjY1OWZm MmJhNTFmMWNmMgogU0laRSAoc2NpbGFiLTUuMi4yLXNyYy50YXIuZ3opID0gNTQwODg1ODAKK01E NSAoamdyYXBoeC0xXzNfMF8wLmphcikgPSBiOWUyNGYwYjQyOGM2Nzk2OTdlNzYzZTliMGJhNTY3 MQorU0hBMjU2IChqZ3JhcGh4LTFfM18wXzAuamFyKSA9IDBmMzcyOGI0N2M5ZmQzOTc3YTA2MzIx MmY2OGU0MzYzOWMyMjgzZWI4NGZiY2U0MzcyNzViNjc5ZTZlNGE2YjIKK1NJWkUgKGpncmFwaHgt MV8zXzBfMC5qYXIpID0gMzI2NTk2CkluZGV4OiBwa2ctcGxpc3QKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQpSQ1MgZmls ZTogL2hvbWUvbmN2cy9wb3J0cy9tYXRoL3NjaWxhYi9wa2ctcGxpc3QsdgpyZXRyaWV2aW5nIHJl dmlzaW9uIDEuMTkKZGlmZiAtdSAtcjEuMTkgcGtnLXBsaXN0Ci0tLSBwa2ctcGxpc3QJMzAgQXBy IDIwMTAgMDg6NTQ6MzkgLTAwMDAJMS4xOQorKysgcGtnLXBsaXN0CTMwIEF1ZyAyMDEwIDAyOjIx OjM3IC0wMDAwCkBAIC01NzA0LDYgKzU3MDQsNyBAQAogJSVHVUklJSUlREFUQURJUiUlL21vZHVs ZXMvamF2YXNjaS90ZXN0cy91bml0X3Rlc3RzL2phdmFjLnNjaQogJSVHVUklJSUlREFUQURJUiUl L21vZHVsZXMvamF2YXNjaS90ZXN0cy91bml0X3Rlc3RzL3Rlc3RfamF2YS5zY2kKICUlR1VJJSUl JURBVEFESVIlJS9tb2R1bGVzL2phdmFzY2kvdmVyc2lvbi54bWwKKyUlR1VJJSUlJURBVEFESVIl JS9tb2R1bGVzL2pncmFwaHguamFyCiAlJURBVEFESVIlJS9tb2R1bGVzL2p2bS9jaGFuZ2Vsb2cu dHh0CiAlJURBVEFESVIlJS9tb2R1bGVzL2p2bS9ldGMvanZtLnF1aXQKICUlREFUQURJUiUlL21v ZHVsZXMvanZtL2V0Yy9qdm0uc3RhcnQKSW5kZXg6IGZpbGVzL3BhdGNoLWV0Yy1saWJyYXJ5cGF0 aC54bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PQpSQ1MgZmlsZTogZmlsZXMvcGF0Y2gtZXRjLWxpYnJhcnlwYXRoLnht bApkaWZmIC1OIGZpbGVzL3BhdGNoLWV0Yy1saWJyYXJ5cGF0aC54bWwKLS0tIC9kZXYvbnVsbAkx IEphbiAxOTcwIDAwOjAwOjAwIC0wMDAwCisrKyBmaWxlcy9wYXRjaC1ldGMtbGlicmFyeXBhdGgu eG1sCTMwIEF1ZyAyMDEwIDAxOjQ3OjM1IC0wMDAwCkBAIC0wLDAgKzEsMjEgQEAKKy0tLSBldGMv bGlicmFyeXBhdGgueG1sLm9yaWcJMjAxMC0wOC0xOCAyMjoxMjo1MS4wMDAwMDAwMDAgLTA3MDAK KysrKyBldGMvbGlicmFyeXBhdGgueG1sCTIwMTAtMDgtMTggMjI6MTU6NDEuMDAwMDAwMDAwIC0w NzAwCitAQCAtMjksMTYgKzI5LDExIEBACisgPHBhdGggdmFsdWU9IiRTQ0lMQUIvbW9kdWxlcy94 cGFkLy5saWJzIi8+CisgPHBhdGggdmFsdWU9IiRTQ0lMQUIvLmxpYnMiLz4KKyAKKy08IS0tIERl ZmF1bHQgcGF0aCB0byB0aGUgSk5JIGNsYXNzZXMgdW5kZXIgc29tZSBMaW51eCAoSm9HTCB1bmRl ciBEZWJpYW4gZm9yIGV4YW1wbGUpIC0tPgorLTxwYXRoIHZhbHVlPSIvdXNyL2xpYi9qbmkvIi8+ CistCisgPCEtLSBNYWtlIGluc3RhbGwgd2l0aCBwcmVmaXggLS0+CisgPHBhdGggdmFsdWU9IiRT Q0lMQUIvLi4vLi4vbGliL3NjaWxhYi8iLz4KKyAKKyA8IS0tIFN0YW5kYXJkIHBhdGggLS0+Cist PHBhdGggdmFsdWU9Ii91c3IvbGliL3NjaWxhYi8iLz4KKy0KKy08IS0tIFN0YW5kYXJkIHBhdGgg Zm9yIDY0IGJpdHMgLS0+CistPHBhdGggdmFsdWU9Ii91c3IvbGliNjQvc2NpbGFiLyIvPgorKzxw YXRoIHZhbHVlPSIlJUpBVkFMSUJESVIlJS8iLz4KKys8cGF0aCB2YWx1ZT0iJSVQUkVGSVglJS9s aWIvIi8+CisgCisgPC9saWJyYXJ5cGF0aHM+Cg== --0016e64717acd1635d048f01ce4e--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTi=hBLL0076RCsOmHq14cU-F5OZ_sjybOcrT-Kho>