Bug 200084

Summary: www/drupal7: [patch] Update Drupal 7.35 to 7.37
Product: Ports & Packages Reporter: Simon Wright <simon.wright>
Component: Individual Port(s)Assignee: Wen Heping <wen>
Status: Closed FIXED    
Severity: Affects Some People CC: wen
Priority: --- Keywords: patch
Version: LatestFlags: simon.wright: maintainer-feedback?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Diff: update Drupal 7.35 to 7.37
none
Diff: update Drupal 7.35 to 7.37
none
Diff: update Drupal 7.35 to 7.37 none

Description Simon Wright 2015-05-09 20:20:13 UTC
Created attachment 156582 [details]
Diff: update Drupal 7.35 to 7.37

Updates Drupal 7.35 to 7.37.

https://www.drupal.org/drupal-7.37-release-notes

No security fixes are included in this release.

Known issues:
A feature of the RESTful Web Services module that allows the return format of an entity to be specified entirely in the URL (for example, accessing http://example.com/node/1.json to view node 1 in JSON format) no longer works in Drupal 7.37. (Previously it worked on some types of databases but not others; for example, it did not work on PostgreSQL databases.) See this issue for further details.
Comment 1 Simon Wright 2015-05-09 21:06:09 UTC
I'm only concerned about the substitution of %%WWWDIR%% for %%DRUPAL_BASE%% in the pkg_plist. This is as suggested by make makeplist. However the files are installed into the correct locations so I'm not sure whether:
- this is correct as per make makeplist
- a change to %%DRUPAL_BASE%% is done automatically when commited or
- I should make this change myself.

Advice please!
Comment 2 Wen Heping freebsd_committer freebsd_triage 2015-05-11 08:09:55 UTC
Hi,

   I have two questions:

   1) why remove lines of @owner www and @group www ?

   2) If you replace %%DRUPAL_BASE%% with %%WWWDIR%%, would you do the same replace for drupal6 , then remove it from Mk/Uses/drupal.mk ?

Regards,
wen
Comment 3 Simon Wright 2015-05-11 20:28:59 UTC
Created attachment 156668 [details]
Diff: update Drupal 7.35 to 7.37

Reverted to original version of the plist except for needed file changes.
Comment 4 Simon Wright 2015-05-11 20:30:29 UTC
Thanks for your comments Wen. I've gone through the original plist again and added the @owner and @group list, plus the missing lines for @dir.

I've also changed the %WWWDIR% back to %DRUPAL_BASE% and attached the updated patch.

I hope this is more acceptable :).
Comment 5 Simon Wright 2015-05-11 20:36:01 UTC
Created attachment 156669 [details]
Diff: update Drupal 7.35 to 7.37

Noticed the use of @sample, updated plist appropriately.
Comment 6 commit-hook freebsd_committer freebsd_triage 2015-05-12 02:41:36 UTC
A commit references this bug:

Author: wen
Date: Tue May 12 02:41:14 UTC 2015
New revision: 386115
URL: https://svnweb.freebsd.org/changeset/ports/386115

Log:
  - Update to 7.37

  PR:		200084
  Submitted by:	simon.wright@gmx.net(maintainer)

Changes:
  head/www/drupal7/Makefile
  head/www/drupal7/distinfo
  head/www/drupal7/pkg-plist