Bug 200084 - www/drupal7: [patch] Update Drupal 7.35 to 7.37
Summary: www/drupal7: [patch] Update Drupal 7.35 to 7.37
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: Wen Heping
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2015-05-09 20:20 UTC by Simon Wright
Modified: 2015-05-12 02:42 UTC (History)
1 user (show)

See Also:
simon.wright: maintainer-feedback?

Attachments
Diff: update Drupal 7.35 to 7.37 (108.16 KB, patch)
2015-05-09 20:20 UTC, Simon Wright 		no flags Details | Diff
Diff: update Drupal 7.35 to 7.37 (9.52 KB, patch)
2015-05-11 20:28 UTC, Simon Wright 		no flags Details | Diff
Diff: update Drupal 7.35 to 7.37 (9.35 KB, patch)
2015-05-11 20:36 UTC, Simon Wright 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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