Bug 206534 - www/py-graphite-web: update to 0.9.15
Summary: www/py-graphite-web: update to 0.9.15
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: Steve Wills
URL:
Keywords: needs-patch, patch, patch-ready
Depends on: 206531 206532
Blocks:
  Show dependency treegraph
 
Reported: 2016-01-23 17:55 UTC by Michael Zhilin
Modified: 2016-02-07 01:57 UTC (History)
2 users (show)

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


Attachments
SVN diff file (11.26 KB, patch)
2016-01-23 17:55 UTC, Michael Zhilin
no flags Details | Diff
poudriere-testport for 10-stable (68.96 KB, text/plain)
2016-01-23 17:56 UTC, Michael Zhilin
no flags Details
Portlint output (12 bytes, text/plain)
2016-01-23 17:56 UTC, Michael Zhilin
no flags Details
SVN diff file (10.79 KB, patch)
2016-01-23 18:20 UTC, Michael Zhilin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Zhilin freebsd_committer freebsd_triage 2016-01-23 17:55:43 UTC
Created attachment 166018 [details]
SVN diff file

Hi,

Here is patch for www/py-graphite-web update to 0.9.15. The main reason is that www/grafana2 recommends to use 0.9.14+ due to performance reasons (http://docs.grafana.org/installation/performance/): 
   
   "Graphite 0.9.14 adds a much needed feature to the JSON rendering API that is very important for Grafana. If you are experiencing slow load & rendering times for large time ranges then it is most likely caused by running Graphite 0.9.12 or lower"

Portlint & testport outputs are attached to this ticket. Also the patches of py-carbon & py-whisper have been submitted as tickets 206531, 206532 (referred as "depends on")

Please review it!

It works for me and my colleagues. 

Thank you!
Comment 1 Michael Zhilin freebsd_committer freebsd_triage 2016-01-23 17:56:39 UTC
Created attachment 166019 [details]
poudriere-testport for 10-stable
Comment 2 Michael Zhilin freebsd_committer freebsd_triage 2016-01-23 17:56:56 UTC
Created attachment 166020 [details]
Portlint output
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2016-01-23 18:00:13 UTC
Please note that www/py-django is:

IGNORE: This port will switch to a meta port soon. Please use py-django18 instead.

If you want this to use a later version of Django (1.9), and you are confident of its compatibility with that versions, then please update the change to use:

www/py-django19
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2016-01-23 18:01:03 UTC
Oops. Forgot to add:

Otherwise leave the dependency at www/py-django18

Thanks again for your submissions Michael
Comment 5 Michael Zhilin freebsd_committer freebsd_triage 2016-01-23 18:20:55 UTC
Created attachment 166021 [details]
SVN diff file

Thank you! 
This is my mistake, it must use django 1.8. 

SVN diff file is updated.
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2016-01-23 19:05:40 UTC
@Michael,

Can you try using USES=shebangfix instead of the s/bash/sh patch
Comment 7 Michael Zhilin freebsd_committer freebsd_triage 2016-01-23 19:11:15 UTC
I'd thought about it, but there are several other lines to be changed in same file. This is interesting point for me. Is it worth to use shebang if file is still require patch?
Comment 8 Kubilay Kocak freebsd_committer freebsd_triage 2016-02-06 07:57:04 UTC
(In reply to Michael Zhilin from comment #7)

Ultimately maintainers decision, but we should do 'the right thing TM'
Comment 9 commit-hook freebsd_committer freebsd_triage 2016-02-07 01:56:32 UTC
A commit references this bug:

Author: swills
Date: Sun Feb  7 01:55:38 UTC 2016
New revision: 408342
URL: https://svnweb.freebsd.org/changeset/ports/408342

Log:
  www/py-graphite-web: update to 0.9.15

  PR:		206534
  Submitted by:	Michael Zhilin <mizhka@gmail.com>

Changes:
  head/www/py-graphite-web/Makefile
  head/www/py-graphite-web/distinfo
  head/www/py-graphite-web/files/patch-conf__graphite.wsgi.example
  head/www/py-graphite-web/files/patch-setup.cfg
  head/www/py-graphite-web/files/patch-setup.py
  head/www/py-graphite-web/files/patch-webapp__graphite__account__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__browser__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__browser__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__cli__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__cli__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__composer__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__dashboard__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__dashboard__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__events__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__events__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__graphlot__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__graphlot__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__local_settings.py.example
  head/www/py-graphite-web/files/patch-webapp__graphite__metrics__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__metrics__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__render__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__render__views.py
  head/www/py-graphite-web/files/patch-webapp__graphite__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__version__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__whitelist__urls.py
  head/www/py-graphite-web/files/patch-webapp__graphite__whitelist__views.py
  head/www/py-graphite-web/files/pkg-message.in
  head/www/py-graphite-web/pkg-plist
Comment 10 Steve Wills freebsd_committer freebsd_triage 2016-02-07 01:57:35 UTC
Committed, with slight modifications, thanks!