Bug 194667 - LibreOffice Tries to install Java 6
Summary: LibreOffice Tries to install Java 6
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-28 17:26 UTC by crum.zach
Modified: 2014-10-30 16:23 UTC (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description crum.zach 2014-10-28 17:26:39 UTC
The Makefile lists Java 1.6 as the required version of Java. 

According to UPDATING
20140627:
  AFFECTS: Users of Java
  AUTHOR: swills@FreeBSD.org

  The default version of OpenJDK has been updated from 1.6 to 1.7. To update,
  users of Java will need to rebuild all ports that depend on Java:
---

By changing JAVA_VERSION= from 1.6 to 1.7, LibreOffice builds without complaining. 

There are many other Java related files required in the JAVA_DISTFILES section, but none of them seem to complain about the change from 1.6 to 1.7
Comment 1 commit-hook freebsd_committer freebsd_triage 2014-10-28 17:49:31 UTC
A commit references this bug:

Author: jkim
Date: Tue Oct 28 17:49:10 UTC 2014
New revision: 371639
URL: https://svnweb.freebsd.org/changeset/ports/371639

Log:
  LibreOffice works fine with Java 7 and later.

  PR:	194667

Changes:
  head/editors/libreoffice/Makefile
Comment 2 Jung-uk Kim freebsd_committer freebsd_triage 2014-10-28 17:52:06 UTC
Fixed in r371639.  Thanks!
Comment 3 Ronald Klop 2014-10-30 16:06:03 UTC
Three lines above the JAVA_VERSION line is also this comment:  # LibreOffice works only with Java 6 -- which may not be the default.
I think it would be proper to also modify/remove that.
Comment 4 Jung-uk Kim freebsd_committer freebsd_triage 2014-10-30 16:23:03 UTC
(In reply to Ronald Klop from comment #3)
> Three lines above the JAVA_VERSION line is also this comment:  # LibreOffice
> works only with Java 6 -- which may not be the default.
> I think it would be proper to also modify/remove that.

Done (r371757).  Thanks for the reminder.