Page MenuHomeFreeBSD
Differential D23725

editors/emacs: Add optional workaround for Fcitx input method
ClosedPublic
Actions

Authored by jrm on Feb 17 2020, 3:00 AM.
Tags
None
Referenced Files
F111381922: D23725.diff
Mon, Mar 3, 2:10 AM
Unknown Object (File)
Tue, Feb 25, 9:46 AM
Unknown Object (File)
Thu, Feb 20, 4:17 AM
Unknown Object (File)
Jan 22 2025, 11:35 AM
Unknown Object (File)
Jan 10 2025, 11:38 AM
Unknown Object (File)
Dec 3 2024, 4:50 PM
Unknown Object (File)
Nov 29 2024, 12:41 AM
Unknown Object (File)
Nov 25 2024, 10:15 AM
View All 96 Files
Subscribers
None

Details

Reviewers
ashish
Commits
rP526478: editors/emacs: Add optional workaround for Fcitx input method
Summary
  • Don't forget to add 'Approved by:' and 'Differential Revision:' **

PR: 242012
Submitted by:
Reported by:

Diff Detail

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

Event Timeline

jrm created this revision.Feb 17 2020, 3:00 AM
Harbormaster completed remote builds in B29437: Diff 68436.Feb 17 2020, 3:00 AM
ashish added a comment.Feb 17 2020, 10:16 AM

I have not built it, but the patch seems to apply fine (including with canna flavour). Should we add an implicit dependency on X11 for FCITXPATCH option, as patch is useless in nox flavour, AFAICS ?

ashish added inline comments.Feb 17 2020, 10:17 AM
editors/emacs/Makefile
115 ↗(On Diff #68436)

Maybe we can add FCITXPATCH here ?

jrm updated this revision to Diff 68451.Feb 17 2020, 11:12 AM

Exclude FCITXPATCH when flavor is nox

Harbormaster completed remote builds in B29443: Diff 68451.Feb 17 2020, 11:12 AM
jrm marked an inline comment as done.Feb 17 2020, 11:13 AM
jrm added inline comments.
editors/emacs/Makefile
115 ↗(On Diff #68436)

I think you're right. Done.

ashish accepted this revision.Feb 18 2020, 7:45 AM

Looks good to me.

Thank you!

This revision is now accepted and ready to land.Feb 18 2020, 7:45 AM
Closed by commit rP526478: editors/emacs: Add optional workaround for Fcitx input method (authored by jrm). · Explain WhyFeb 18 2020, 10:57 PM
This revision was automatically updated to reflect the committed changes.
jrm marked an inline comment as done.
jrm added a commit: rP526478: editors/emacs: Add optional workaround for Fcitx input method.

Revision Contents
Changeset List

PathSize
head/
editors/
emacs/
19 lines
files/
12 lines

Diff 68509

View Options

head/editors/emacs/Makefile

Loading...
View Options

head/editors/emacs/files/extra-patch-src_xfns.c

Loading...