Stop hardcoding the YACC error message
ClosedPublic
Actions

Authored by alexmv on Jan 12 2017, 3:26 PM.

Tags

None

Referenced Files

	F18109808: D17189.id41334.diff
	Mon, Aug 11, 3:20 PM

	F18109807: D17189.id.diff
	Mon, Aug 11, 3:20 PM

	F18104705: D17189.diff
	Sun, Aug 10, 1:40 PM

	F18093472: D17189.id41334.diff
	Thu, Aug 7, 6:37 PM

	F17948173: D17189.id.diff
	Jul 31 2025, 8:17 PM

	F17928552: D17189.id.diff
	Jul 30 2025, 4:16 PM

	F17919721: D17189.diff
	Jul 30 2025, 2:51 AM

	F17906687: D17189.diff
	Jul 29 2025, 12:44 PM

View All Files

Subscribers

epriestley

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUe4b06e06ee78: Stop hardcoding the YACC error message

Summary

The exact syntax of the error varies with YACC version.

Ref D17188#204123.

Test Plan

Unit tests.

Diff Detail

Repository

rPHU libphutil

Branch

up-yacc-iniparser

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 15221
Build 20007: Run Core Tests
Build 20006: arc lint + arc unit

Event Timeline

alexmv created this revision.Jan 12 2017, 3:26 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 12 2017, 3:26 PM

Herald added a subscriber: epriestley. · View Herald Transcript

Harbormaster failed remote builds in B15221: Diff 41334!Jan 12 2017, 3:26 PM

Harbormaster failed remote builds in B15221: Diff 41334!

Oh, this didn't push to staging which is why the build isn't working; T10093. Change is fine anyway.

This revision is now accepted and ready to land.Jan 12 2017, 3:27 PM

alexmv closed this revision.Jan 12 2017, 3:30 PM

Revision Contents
Changeset List

Path

Size

src/

utils/

__tests__/

PhutilUtilsTestCase.php

6 lines

Diff 41334

View Options

src/utils/tests/PhutilUtilsTestCase.php

Stop hardcoding the YACC error messageClosedPublicActions