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)
Mon, May 5, 6:46 PM
Unknown Object (File)
Tue, Apr 29, 2:20 PM
Unknown Object (File)
Wed, Apr 23, 1:19 PM
Unknown Object (File)
Mon, Apr 21, 4:22 AM
Unknown Object (File)
Fri, Apr 18, 2:44 PM
Unknown Object (File)
Thu, Apr 17, 7:15 AM
Unknown Object (File)
Apr 2 2025, 7:38 PM
Unknown Object (File)
Mar 18 2025, 11:00 PM
View All 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...