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

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 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 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