Paths

Table of Contentst

.hooks/pre-commit.d: New hook to validate vuxml
ClosedPublic
Actions

Authored by grembo on Tue, Nov 3, 1:37 PM.

Project Tags

None

Referenced Files

	F177467780: D61157.1794495434.diff
	Wed, Nov 11, 2:57 PM

	F177465903: D61157.1794490322.diff
	Wed, Nov 11, 1:32 PM

Subscribers

dvl

mat

Details

Reviewers

joneum
mat

Group Reviewers

ports secteam
portmgr

Commits

R11:12edba187331: .hooks/pre-commit.d: New hook to validate vuxml

Summary

This runs make validate in case VuXML entries have been changed.

Test Plan

Modify security/vuxml/vuln/2026.xml, run git commit.

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 78688
Build 75571: arc lint + arc unit

Event Timeline

grembo requested review of this revision.Tue, Nov 3, 1:37 PM

grembo created this revision.

Harbormaster completed remote builds in B78688: Diff 191385.Tue, Nov 3, 1:37 PM

grembo added reviewers: ports secteam, portmgr.Tue, Nov 3, 1:40 PM

Remove unneeded include

Harbormaster completed remote builds in B78689: Diff 191386.Tue, Nov 3, 1:40 PM

mat subscribed.Tue, Nov 3, 6:48 PM

mat added inline comments.

.hooks/pre-commit.d/validate_vuxml
11

FreshPorts and I will thank you as I will have fewer failure messages to relay.

Incorporate feedback

Harbormaster completed remote builds in B78695: Diff 191411.Tue, Nov 3, 7:31 PM

git root is probably a local alias for git rev-parse --show-toplevel?

joneum accepted this revision.Tue, Nov 3, 7:42 PM

This revision is now accepted and ready to land.Tue, Nov 3, 7:42 PM

In D61157#1402361, @grembo wrote:

git root is probably a local alias for git rev-parse --show-toplevel?

Ah, no, sorry, it comes with git-extras, but you are correct, it does git rev-parse --show-toplevel.

mat accepted this revision.Wed, Nov 4, 8:53 AM

Closed by commit R11:12edba187331: .hooks/pre-commit.d: New hook to validate vuxml. · Explain WhyWed, Nov 4, 10:50 AM

This revision was automatically updated to reflect the committed changes.

grembo added a commit: R11:12edba187331: .hooks/pre-commit.d: New hook to validate vuxml.

Revision Contents
Changeset List

Path

Size

.hooks/

pre-commit.d/

validate_vuxml

17 lines

Diff 191385

View Options

.hooks/pre-commit.d/validate_vuxml

.hooks/pre-commit.d: New hook to validate vuxmlClosedPublicActions