Bug 219131 - www/grafana4: wrong boot.js installed
Summary: www/grafana4: wrong boot.js installed
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: Steve Wills
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-08 05:52 UTC by Ari Suutari
Modified: 2017-05-31 17:04 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (swills)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ari Suutari 2017-05-08 05:52:20 UTC
www/grafana4 port installs wrong boot.js file. The resulting installation works, but performs poorly, because it is using development version of boot.js, which causes many javascript files to be downloaded separately. This causes considerable delay to startup time in browser when using the system over slower link.

Port builds installation from two distfiles: grafana-4.2.0.linux-x64.tar.gz and grafana-grafana-v4.2.0_GH0.tar.gz. The linux version contains the correct deployment version of boot.js and the later one has the development version. 

Unfortunately the linux distribution is unpacked first and ...._GH0.tar.gz after that, which overwrites to deployment version of boot.js.

Not a solution, but a workaround is to remove /usr/local/share/grafana/public/app/boot.js after installation and link boot.5ee7468a.js to it.
Comment 1 commit-hook freebsd_committer freebsd_triage 2017-05-31 16:53:19 UTC
A commit references this bug:

Author: swills
Date: Wed May 31 16:53:04 UTC 2017
New revision: 442223
URL: https://svnweb.freebsd.org/changeset/ports/442223

Log:
  www/grafana4: update and fix bugs

  * Update to 4.3.2 [1]
  * Fix paths not to include the major version to fix plugin install and keep things uniform [2]
  * Correct boot.js seems to be used now [3]

  PR:		219639 [1]
  PR:		218010 [2]
  PR:		219131 [3]
  Submitted by:	farrokhi [1]
  Reported by:	Borja Marcos <borjam@sarenet.es> [2]
  Reported by:	Ari Suutari <ari@stonepile.fi> [3]

Changes:
  head/www/grafana4/Makefile
  head/www/grafana4/distinfo
  head/www/grafana4/files/grafana.conf.in
  head/www/grafana4/files/grafana.in
  head/www/grafana4/files/grafana4.conf.in
  head/www/grafana4/files/grafana4.in
  head/www/grafana4/pkg-plist
Comment 2 Steve Wills freebsd_committer freebsd_triage 2017-05-31 17:04:40 UTC
Seems to be correct in the new version, let me know if not.