19:06 Rene Ladan (rene) 

Tools/scripts: remove addport

The addport script is no longer useful since git has been introduced in
April 2021, just use plain "git add" instead. Do not forget to add the
port name to the category Makefile and mention the contributor in the
contributors article in the doc repository.

Differential Revision: https://reviews.freebsd.org/D35104

    bec21c9

14:27 Mathieu Arnold (mat) 

framework: Remove $FreeBSD$

Where appropriate fiddle with a few other things.

    5d33e04

12:37 emaste 

addport: Update contributor list file after Asciidoc conversion

Reviewed by:	crees
Approved by:	crees
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D29476

 

13:54 pizzamig 

Tools/script/addport: Fix usage with multiple directories

PR:		238379
Submitted by:	me
Approved by:	crees@

 

12:49 mat 

Use make(1) to extract variables from a Makefile, not grep.

PR:		212486
Submitted by:	mat
Approved by:	crees (maintainer)
Sponsored by:	Absolight

 

09:14 matthew 

Fix the '-a' option to ${PORTSDIR}/Tools/scripts/addport

Summary:
Currently addport -a ends up running a command line like:

   make  PORTSDIR="/tmp/ap.BsOuZJBy" clean check-categories

where the temporary directory has only parts of the ports tree
(specifically bits under ${PORTSDIR}/Mk) checked out.  This can't work
as 'make check-categories' needs to see the entire ports tree checked
out.

Instead, run the checks specified by '-a' and '-t' with the value of
$PORTSDIR from the environment.

Reviewers: crees

Reviewed By: crees

Subscribers: mat

Differential Revision: https://reviews.freebsd.org/D7525

 

19:29 rm 

Let addport script to use svnlite if available

1. Check if svnlite is available and use it by default, else: use svn
2. Dehardcode svn binary name in shell calls, to make both svn/svnlite
work

Approved by:	crees (maintainer)

 

15:31 naddy 

Default to repo.FreeBSD.org for commits, as urged by our repo meister.

Approved by:	crees

 

13:38 mat 

Don't add ports/ to the PR line.

Sponsored by:	Absolight

 

19:20 crees 

Require versions of perl supporting smartmatch operator (~~)

Submitted by:	az

 

16:17 crees 

Fix -c option

PR:		ports/178193
Submitted by:	tota

 

16:13 crees 

Use MAINTAINER line for Submitted by: if Created by: does not exist.

Still prefer Created by because we still don't use real
names in MAINTAINER lines (oh please let that change....)

Submitted by:	jgh (based on)

 

17:40 crees 

Don't hardcode make

Submitted by:	az

 

16:55 crees 

Apparently some people override PKGCATEGORY, so
manually extract it from CATEGORIES again.

Discovered by:	dinoex

 

09:51 crees 

Add some checks for Subversion; minimum version required is 1.7, or adding
ports with subdirectories fails because of scattered .svn directories.

Original issue reported by:	netchild

 

18:59 crees 

Strip svn:executable just in case it's been set

Reported by:	db

 

11:37 crees 

Freefall doesn't have LWP::Simple, so use /usr/bin/fetch to reduce dependencies

Noticed by:	thierry
Feature safe:	yes

 

21:30 madpilot 

- Fix reference to the Additional Contributors' list due to renaming
  to .xml in r39631

Approved by:	crees
Feature safe:	yes

 

15:55 ale 

Clean-up command-line options.

Approved by:	crees

 

19:25 crees 

bsd.port.mk handles getting the primary category for us

Since Created by: line appears to be popular, get Submitted by: from that

Old-style Makefile headers should no longer be used

 

13:31 crees 

Accepts the new svn_revision tag in the removed ports xml database.

Apologies for the complexity of the code; it's optimised for speed.  Soon I
will be able to remove the section working with dates.

 

18:40 crees 

Now that the headers have been removed, we no longer get the Whom line from
which to acquire the Submitted by: line.

It should be reasonable to simply use the MAINTAINER line; unfortunately we
lose the real name and only get the email address.

 

12:02 crees 

Fix the -y option to test for possible duplicate ports; use INDEX instead of
modules file that was removed.

It's fast, use it!

10:20 crees 

- Look for older versions of ports to copy over before committing.

Requested by:	several

- Remove older readded ports from MOVED

- Tighten up find syntax (split by null and use -print0)

10:49 crees 

Don't replace - with _ in category variable.

This was required with CVS, since the category was checked out as a module, but
the directory name is used with svn.

For example:

pcvs co ports_ports_mgmt

svn co $ports/head/ports-mgmt

Submitted by:	swills

20:57 svn2cvs 

SVN rev 300901 on 2012-07-15 20:57:22Z by beat

- Update addport and rmport to work with Subversion

Submitted by:   crees, miwi

20:57 beat 

- Update addport and rmport to work with Subversion

Submitted by:	crees, miwi

18:54 crees 

- Modify Additional Contributors path to match svn

- Take maintainership

Approved by:    garga (maintainer)

14:54 crees 

Add option (-y) to check for similarly named ports.

PR:             ports/159841
Submitted by:   crees (me)
Approved by:    garga (maintainer)

00:09 swills 

- Back out category check for now

21:21 swills 

- Remove obfuscation of submitter email at request of dougb
- Add check that port is in right directory as I've been bitten by this

17:18 garga 

Change shebang to #!/usr/bin/env perl, as required by bapt@

09:30 romain 

- Unbreak the -n flag;
- Remove invalid reference to freefall.

Approved by:    garga

15:44 dinoex 

- allow addport to work in ports-mgnt again
Approved by:    garga, drso, flz

21:32 sobomax 

Make sure we don't add ${WRKDIR}.

22:09 erwin 

CVSROOT/modules no longer contains an entry per port, remove support
for this from these scripts.

16:54 garga 

Fix a problem when add new ports on ports-mgmt category. ports-mgmt/Makefile
lines has 3 spaces before SUBDIR word and all other categories has 4.

I've asked pav@ if there is a default format of category Makefiles and he said
the number of spaces doesn't matter, so, i fix addport to respect the current
number of spaces and/or tabs the file has.

Reported by:    miwi, erwin

12:45 garga 

Fix addport to work correctly with ports-mgmt category.

Reported by:    miwi
Tested by:      miwi

01:49 shaun 

Add an option to the addport script: -b requests that any files in
${FILESDIR} which look like patches be treated as binary files. This
prevents RCS tags in patch fragments causing a problem for CVS.

Approved by:    garga (maintainer),
                ahze (mentor, implicit)

23:12 garga 

When automatic commit log is generated, change @ to " at " on submitter email
to prevent spam

15:44 garga 

- Change addport script to do the job locally instead connect to freefall every
  time to add a new module. If you want to still use old way, just use
  "-M freefall.FreeBSD.org" option
- Take addport maintainership
- When modulesupdate fail, ask user to retry
- Change modulesupdate to work fine with addport

Approved by:    will (maintainer)

16:26 garga 

- Add a new option -L, it works like -l but uses pkg-descr to create
commit message.

Approved by:    will (maintainer)

22:18 marcus 

Add prototypes for all the subroutines which fixes compilation with
Perl 5.8.7.

Reported by:    mezz

11:03 vs 

As discussed on the cvs-ports-list, make 'addport' print a warning if the
directory
name contains upper-case characters as it makes finding a port harder.

PR:             ports/79679
Approved by:    will, portmgr(krion)

17:32 vs 

Checkout Templates from CVS as well since ports with GNU_CONFIGURE will
need them when building from 'addport -a -t'.

Approved by: will

10:21 krion 

Remove traling spaces.

04:09 dd 

Newline should go after the terminating punctuation (fix typo).

Approved by:    will

21:10 mat 

While generating commit message, don't lowercase the first letter if the second
is upper case (ie : don't end up with pEAR)

Approved by:    will
Asked by:       thierry

18:52 mat 

Fix a bug arising when you added a port which should have been at the end of
<category>/Makefile, and ended up at the begining

Approved by:    will

07:28 ale 

- Fix synopsis
- Add 'ports/' PR category to commit log

Approved by:    will

20:03 demon 

When suggesting to add new port's submitter to Contributors' List,
also print the exact location of the file to checkout.
This is for those who rarely adds new ports and do not remember the
location of Additional FreeBSD Contributors article.

Approved by:    will

06:04 petef 

Refactor how we format the autofill message.  With the slightly questionable
Perl before, sometimes we would word wrap too soon.  Now we should never
generate a line more than 72 characters long (but as close to 72 characters
long as we can without breaking up a word).

Approved by:    will (maintainer)

15:09 des 

fetch(1)'s -b and -t options were deprecated years ago, so stop using them.

02:39 bland 

Fix -l option comment removing obsolete pkg-comment reference.

Approved by: will

15:33 mat 

When using addport on freefall, there's no need to ssh freefall once more to
use modulesupdate.

Approved by:    will

19:49 peter 

Update for post-repo-split.  s/ncvs/pcvs/ so that the default root is
pcvs.freebsd.org:/home/pcvs

20:06 lioux 

Another try at fixing the "adding to CVSROOT/modules from remote
hosts (non-freefall)" bug: add '-A' argument to ssh(1) command line
for accessing host cointaining modules file. This should fix the
bug for ssh-agent(1) users. Others can just switch to using
ssh-agent(1).

Approved by:    will (maintainer)

15:23 nork 

Change how to get CATEGORIES tag from new port for slave port.

Approved by:    will

08:56 kris 

Catch up to new semantics of COMMENT variable (contains comment string
itself, instead of pointing to pkg-comment)

17:28 will 

Attempt to defuse problems from developers who don't have the correct
ssh setup in order for addport to do CVSROOT/modules correctly.

07:05 lioux 

Add both hungarian and portuguese l10n prefixes to syncronize with
portlint port

Approved by:    maintainer

14:30 alane 

unbroke addport by petef

15:00 petef 

Use 'cvs ci -F ...' instead of setting the log file with EDITOR="cp ...".
This fixes addport for me on -stable (cvs was erroring out), and also
fixes addport when you have the EDITOR environment variable set.

Approved by:    will

08:45 will 

Modifications to support ncvs.freebsd.org != freefall.freebsd.org.  This
actually simplifies some of the code and makes other parts more complicated.

Tested by:      www/mod_filter && (after bugfixes) sysutils/sjog
Noticed by:     ijliao (by way of peter)

00:05 perky 

Add l10n prefix to default module name.

Approved by: will