Bug 199399 - math/geogebra-i18n: Update to latest version
Summary: math/geogebra-i18n: Update to latest version
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Kurt Jaeger
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2015-04-12 15:43 UTC by Zsolt Udvari
Modified: 2016-03-09 18:00 UTC (History)
4 users (show)

See Also:
pi: maintainer-feedback+


Attachments
SHAR of directory structure (45.18 KB, text/plain)
2015-04-12 15:44 UTC, Zsolt Udvari
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Zsolt Udvari freebsd_committer freebsd_triage 2015-04-12 15:43:47 UTC
I think maybe should new port instead of update math/geogebra-i18n because this version is a huge package (java-included) with dependencies of linux emulation.

If would better to update the existing math/geogebra-i18n please let me know and I'll change the Makefile.
Comment 1 Zsolt Udvari freebsd_committer freebsd_triage 2015-04-12 15:44:28 UTC
Created attachment 155520 [details]
SHAR of directory structure
Comment 2 Kurt Jaeger freebsd_committer freebsd_triage 2016-02-27 22:50:50 UTC
It would be better to update math/geogebra-i18n, can you please submit a patch to the old port instead of a new port ?
Comment 3 Zsolt Udvari freebsd_committer freebsd_triage 2016-02-28 08:08:48 UTC
I thought that the "i18n" suffix isn't clear and I don't know why it need.
But if you think the update would be better I'll make it of course (but not now, only some days later).
Comment 4 Kurt Jaeger freebsd_committer freebsd_triage 2016-02-28 12:35:57 UTC
Send a mail to submitter and maintainer if port rename to geogebra without the -i18n prefix would be appropriate. Pending results.
Comment 5 Nicola Vitale freebsd_committer freebsd_triage 2016-02-28 14:26:32 UTC
(In reply to Zsolt Udvari from comment #3)

The suffix -i18n is an old convention (see par. 15.3.3 https://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/l10n-compiling.html ); we can remove it without problem in my opinion.
Comment 6 Kurt Jaeger freebsd_committer freebsd_triage 2016-03-06 16:22:07 UTC
testbuilds@work
Comment 7 Zsolt Udvari freebsd_committer freebsd_triage 2016-03-06 16:28:43 UTC
On the next week I'll have some free time so I can update this update. If you can wait some days I'll attach poudriere log and test the port.
Comment 8 Kurt Jaeger freebsd_committer freebsd_triage 2016-03-06 16:29:12 UTC
testbuilds are fine.
Comment 9 commit-hook freebsd_committer freebsd_triage 2016-03-06 16:58:30 UTC
A commit references this bug:

Author: pi
Date: Sun Mar  6 16:58:20 UTC 2016
New revision: 410466
URL: https://svnweb.freebsd.org/changeset/ports/410466

Log:
  math/geogebra-i18n: rename to math/geogebra, 4.2.47.0 -> 5.0.208.0

  Changes:
    http://www.geogebra.org/wiki/en/Reference:Changelog_5.0

  PR:		199399
  Submitted by:	Zsolt Udvari <udvzsolt@gmail.com>
  Approved by:	nivit

Changes:
  head/MOVED
  head/math/geogebra/
  head/math/geogebra/Makefile
  head/math/geogebra/distinfo
  head/math/geogebra/files/geogebra.desktop.in
  head/math/geogebra/files/patch-geogebra
  head/math/geogebra/pkg-plist
  head/math/geogebra-i18n/
Comment 10 Kurt Jaeger freebsd_committer freebsd_triage 2016-03-06 16:59:38 UTC
Committed, thanks.
Comment 11 John Marino freebsd_committer freebsd_triage 2016-03-09 09:52:58 UTC
doesn't this needs USES=LINUX ?

RUN_DEPENDS=    bash:${PORTSDIR}/shells/bash \ /compat/linux/bin/sh:${PORTSDIR}/emulators/linux_base-c6

I don't know where this run depends on linux_base-c6 comes from but if this is a linux port, why is it differing from every other one?

or is this some kind of unholy monster that is a native application that requires "sh" from linux emulation?
Comment 12 John Marino freebsd_committer freebsd_triage 2016-03-09 09:56:42 UTC
i mean USE_LINUX=yes or somethign like that.
Comment 13 Kurt Jaeger freebsd_committer freebsd_triage 2016-03-09 10:20:24 UTC
(In reply to John Marino from comment #12)
Good question, I'll ping the maintainer.
Comment 14 Zsolt Udvari freebsd_committer freebsd_triage 2016-03-09 10:51:44 UTC
I checked the Uses/bsd.linux-apps.mk:
- USE_LINUX=yes is a stupid thing: value of USE_LINUX will the SUFFIX of linux-packages (linux_base-${USE_LINUX}, now the c6 is the default)
- USE_LINUX_APPS should be the good variable but as I see can't add "linux_base" (linux_base will e.g. the dependency of linux_cairo)
- I'll check some hours later the other runtime-dependency (linux_cairo or similar)
Comment 15 Zsolt Udvari freebsd_committer freebsd_triage 2016-03-09 18:00:52 UTC
I've created a patch and bug report, please check: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=207852