MOVEDlint.awk: add the ability to select a start date
ClosedPublic
Actions

Authored by bapt on Aug 23 2022, 9:22 AM.

Details

Reviewers

tcberner

Commits

R11:d2aa049c8610: MOVEDlint.awk: add the ability to select a start date

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 47068
Build 43957: arc lint + arc unit

Event Timeline

bapt created this revision.Aug 23 2022, 9:22 AM

Herald added a subscriber: mat. · View Herald TranscriptAug 23 2022, 9:22 AM

bapt requested review of this revision.Aug 23 2022, 9:22 AM

Harbormaster completed remote builds in B47067: Diff 109703.Aug 23 2022, 9:22 AM

tcberner added a subscriber: tcberner.Aug 23 2022, 9:25 AM

tcberner added inline comments.

Tools/scripts/MOVEDlint.awk
47–52	^ how about sourcing it from `ENVIRON[LASTDATE]` instead? lastdate=ENVIRON["LASTDATE"] ? ENVIRON["LASTDATE"] : "1999-12-31"

tcberner added inline comments.Aug 23 2022, 9:32 AM

Tools/scripts/MOVEDlint.awk
63	This should likely be !started && $3 >= lastdate {

fix last date checking logic

Harbormaster completed remote builds in B47068: Diff 109704.Aug 23 2022, 9:58 AM

lgtm -- I'll create a git hook, after you push it (and MOVED is free of errors).

This revision is now accepted and ready to land.Aug 23 2022, 10:16 AM

Closed by commit R11:d2aa049c8610: MOVEDlint.awk: add the ability to select a start date (authored by bapt). · Explain WhyAug 23 2022, 12:08 PM

This revision was automatically updated to reflect the committed changes.

bapt added a commit: R11:d2aa049c8610: MOVEDlint.awk: add the ability to select a start date.

Revision Contents
Changeset List

Path

Size

Tools/

scripts/

MOVEDlint.awk

15 lines

Diff 109704

View Options

Tools/scripts/MOVEDlint.awk

MOVEDlint.awk: add the ability to select a start dateClosedPublicActions