Page MenuHomeFreeBSD

editors/emacs-devel: Update to latest commit on master branch.
ClosedPublic

Authored by jrm on Jan 22 2017, 3:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 23 2024, 7:19 AM
Unknown Object (File)
Feb 22 2024, 3:34 PM
Unknown Object (File)
Jan 31 2024, 3:47 PM
Unknown Object (File)
Jan 28 2024, 5:10 AM
Unknown Object (File)
Dec 30 2023, 1:07 PM
Unknown Object (File)
Dec 30 2023, 1:06 PM
Unknown Object (File)
Dec 20 2023, 8:28 AM
Unknown Object (File)
Oct 14 2023, 7:30 AM
Subscribers

Details

Summary

editors/emacs-devel: Update to latest commit on master branch.

  • Switch to using the supplied autoconf.sh, because it is required on 10.3 to create nt/gnulib.mk.
  • Patch to accept makeinfo versions as far back as 4.8, which is the version in 10.3 base.
Test Plan

portlint -C: OK
testport: OK (poudriere: 10.3-RELEASE-p10, i386, default options)
testport: OK (poudriere: 10.3-RELEASE-p10, amd64, default options)
testport: OK (poudriere: 11.0-RELEASE-p1, i386, default options)
testport: OK (poudriere: 11.0-RELEASE-p1, amd64, default options)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 6975
Build 7160: arc lint + arc unit

Event Timeline

jrm retitled this revision from to editors/emacs-devel: Update to latest commit on master branch..
jrm updated this object.
jrm edited the test plan for this revision. (Show Details)
jrm added reviewers: swills, AMDmi3, ashish.
swills edited edge metadata.

Approved

This revision is now accepted and ready to land.Jan 23 2017, 1:08 PM

I will wait for your approval ashish. In the future, how would you feel about implicit approval for updates that only include a PORTREVISION bump? For the development version, I feel it's important to have frequent updates, so new bugs in the master branch are discovered as early as possible.

The diff looks good, although xwidgets option is broken, because it relies on a later version of WebKit GTK3 which is not yet in ports tree, so might want to add a IGNORE clause for that, or just remove it for now.

In D9287#192339, @jrm wrote:

implicit approval for updates that only include a PORTREVISION bump? For the development version, I feel it's important to have frequent updates, so new bugs in the master branch are discovered as early as possible.

Sure, for PORTREVISION bump, but this update is not that.

jrm edited edge metadata.

Set IGNORE when the XWIDGETS option is turned on. Seeing the IGNORE message may motivate some WebKit GTK3 work.

This revision now requires review to proceed.Jan 24 2017, 2:44 AM

Agreed. This diff would not qualify. If even a single line other than the PORTVERSION is changed in the Makefile, the normal approval process would apply.

jrm edited edge metadata.

Tweak the IGNORE message when the XWIDGETS option is selected.

Remove period at the end of the IGNORE message.

ashish edited edge metadata.

Looks good to me. Please proceed.

Thanks!

This revision is now accepted and ready to land.Jan 24 2017, 5:59 AM
This revision was automatically updated to reflect the committed changes.