10:00 Tobias C. Berner (tcberner) 

framework: cleanup conditional-indentations in Mk/

Run Tools/scripts/indent_make_if.pl on all of Mk.

These white space changes contribute greatly to the readability of those files.
As we have a version control system, finding out the reasons for the changes
prior to these white space changes is still easily possible

Differential Revision:	https://reviews.freebsd.org/D35024
Reviewed by:		portmgr (rene, bapt)

    aa25396

14:27 Mathieu Arnold (mat) 

framework: Remove $FreeBSD$

Where appropriate fiddle with a few other things.

    5d33e04

08:22 rene 

Mk/Uses/drupal.mk: remove support for Drupal 6

 

01:23 bapt 

Remove the now unneeded ${PORTSDIR} from dependency definition in
The infrastructure Makefiles

PR:		206569
Exp run by:	antoine
Differential Revision:	D5047

 

08:51 marino 

Remove OPTIONS_DEFINE out of Mk/Uses/drupal.mk (affects many drupal ports)

The USES= fragments are not supposed to modify OPTIONS_* because they are
loaded after bsd.options.mk.

In the particular case of drupal, this resulted in SELECTED_OPTIONS and
DESELECTED_OPTIONS being incorrect.  A second problem was that the "="
was used for assignment rather than "?=", meaning that any port with
USES=drupal got their options overwritten at some point (this included
the main www/drupal6 and www/drupal7 ports).

This commit adds OPTIONS_DEFINE=DOCS to almost every port that had set
USES=drupal to correct the mistake of setting options in Mk/Uses.

PR:	206060

 

08:07 miwi 

- Release maintainership

 

07:36 bapt 

Readd PORTSDIR for now we will only start removing them after 2016Q1 is branched

This gives more time for tools to get updated, available in packages etc before
bothering users

 

16:49 bapt 

Drop the necessity to add ${PORTSDIR} to dependency line

Modify make describe to automatically prepend ${PORTSDIR} if the path for the
port is not absolute

Checked with poudriere, portmaster, portupgrade

PR:		203685
Exp-run by:	antoine
Differential Revision:	https://reviews.freebsd.org/D3866

 

17:05 antoine 

Cleanup plist a bit, DRUPAL_MODDIR and DRUPAL_BASE belong to drupal ports,
not to drupal modules

With hat:	portmgr

 

21:35 antoine 

wheel is a group, not an owner

With hat:	portmgr

 

16:36 tijl 

Change the way USES is handled:
- Loop over USES twice, once to define all *_ARGS variables and once to
  include Uses/*.mk.  This allows all Uses/*.mk to examine arguments given
  to other USES entries.
- Always define *_ARGS (possibly empty) and replace commas with spaces.

Similar for _USES_POST.

Adjust all Uses/*.mk:
- defined(u_ARGS) becomes !empty(u_ARGS)
- Eliminate helper variables like _*_ARGS=${*_ARGS:C/,/ /g}
- Some Uses/*.mk used ":" as argument separator instead of ",", but no port
  used this form
- Uses/cran.mk: remove unused variable VALID_ARGS and USES+=fortran which
  has no effect
- Uses/twisted.mk: simplify handling of the case where neither "build" nor
  "run" arguments have been specified

PR:		193931
Exp-run by:	antoine
Approved by:	portmgr (antoine)

 

09:50 antoine 

Do not override _USES_POST

Reviewed by:	bapt
Phabric:	D560
With hat:	portmgr

 

12:16 bapt 

New USES=drupal
It accepts 6 or 7 as a mandatory argument (version of drupal)
It accept nothing (aka drupal main port) or a mutually exclusive
"theme"/"module"