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: | Latest | Flags: | simon.wright:
maintainer-feedback?
|
||||||||
Hardware: | Any | ||||||||||
OS: | Any | ||||||||||
Attachments: |
|
Description
Simon Wright
2015-05-09 20:20:13 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! 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 Created attachment 156668 [details]
Diff: update Drupal 7.35 to 7.37
Reverted to original version of the plist except for needed file changes.
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 :). Created attachment 156669 [details]
Diff: update Drupal 7.35 to 7.37
Noticed the use of @sample, updated plist appropriately.
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 |