Page MenuHomeFreeBSD
Differential D25244

lang/ecl: Fix r538595
ClosedPublic
Actions

Authored by salvadore on Jun 12 2020, 11:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 9, 5:06 PM
Unknown Object (File)
Thu, May 9, 5:06 PM
Unknown Object (File)
Thu, May 9, 5:06 PM
Unknown Object (File)
Thu, May 9, 5:06 PM
Unknown Object (File)
Sun, May 5, 12:43 AM
Unknown Object (File)
Sat, Apr 20, 5:51 PM
Unknown Object (File)
Apr 4 2024, 5:53 PM
Unknown Object (File)
Mar 5 2024, 7:45 PM
View All 47 Files
Subscribers
None

Details

Reviewers
mat
thierry
olgeni
gerald
tcberner
Commits
rP538656: lang/ecl: Fix r538595
Summary

Move the comment introduced in r538595 so that it does not break the port anymore.

Reported by: mat

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

salvadore requested review of this revision.Jun 12 2020, 11:39 AM
salvadore created this revision.
Harbormaster completed remote builds in B31661: Diff 73018.Jun 12 2020, 11:39 AM
salvadore added a comment.Jun 12 2020, 11:40 AM

This time I have tested it properly in a poudriere jail. (Only one, I hope it's enough.)

salvadore added a comment.Jun 12 2020, 12:18 PM

Sorry, I forgot about a warning from mat: line 11 is now part of the comment started at line 10.
I have to look at this review again.

salvadore updated this revision to Diff 73019.Jun 12 2020, 12:52 PM

I moved the comment from within the patch to the top of it: it is a different style than the one used by Thierry for the other patches he introduced, but I think it makes more sense in this case.

Tested successfully with poudriee i386 11.3-RELEASE.

Harbormaster completed remote builds in B31662: Diff 73019.Jun 12 2020, 12:52 PM
mat added a comment.Jun 12 2020, 3:36 PM

looks ok to me

tcberner accepted this revision.Jun 13 2020, 9:19 AM

lgtm

This revision is now accepted and ready to land.Jun 13 2020, 9:19 AM
Closed by commit rP538656: lang/ecl: Fix r538595 (authored by salvadore). · Explain WhyJun 13 2020, 9:26 AM
This revision was automatically updated to reflect the committed changes.
salvadore added a commit: rP538656: lang/ecl: Fix r538595.

Revision Contents
Changeset List

PathSize
head/
lang/
ecl/
files/
4 lines

Diff 73063

View Options

head/lang/ecl/files/extra-patch-src_Makefile.in

Loading...