Bug 1230278 - color overstrike and color underline in .mostrc have no effect
Summary: color overstrike and color underline in .mostrc have no effect
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: most
Version: 22
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Adrian Reber
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-06-10 14:23 UTC by Van de Bugger
Modified: 2015-12-01 19:51 UTC (History)
1 user (show)

Fixed In Version: most-5.0.0-12.a.1.fc23 most-5.0.0-12.a.1.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-01 17:53:05 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Fix for the problem. (329 bytes, patch)
2015-06-10 14:23 UTC, Van de Bugger
no flags Details | Diff

Description Van de Bugger 2015-06-10 14:23:30 UTC
Created attachment 1037330 [details]
Fix for the problem.

Description of problem:

Specifying overstrike and underline colors in .mostrc have no effect.

Version-Release number of selected component (if applicable):

Fedora 22 on x86_64
most-5.0.0-10.a.1.fc22.x86_64

How reproducible:

Always

Steps to Reproduce:
1. dnf install most
2. echo 'normal brightgray black' > ~/.mostrc
3. echo 'color underline cyan black' >> ~/.mostrc
4. echo 'color overstrike yellow black' >> ~/.mostrc
5. echo 'color status black gray' >> ~/.mostrc
6. PAGER=most man most

Actual results:

Man page shown. Normal text light gray on black. Section headers ("NAME", "SYNOPSIS", "DESCRIPTION", others) are in red-on-black color. Program name ("most" in the text) is in green-on-black color. Status line is in black -on gray color.

Expected results:

Section headers ("NAME", "SYNOPSIS", "DESCRIPTION", others) are in yellow-on-black color. Program name ("most" in the text) is in cyan-on-black color.

Additional info:

See attached patch one-line. It may be implemented wrongly (or not completely — I have no idea about internal architecture of most), but it works for me. Note most_setup_colors function (line 285 of keyparse.c) — when setting up default colors, SLtt_set_color is called twice:

>    SLtt_set_color (h->value, NULL, h->fg, h->bg);
>    SLtt_set_color (h->value+MOST_EMBEDDED_COLOR_OFFSET, NULL, h->fg, h->bg);

And now see set_object_color function (line 206 of the same file) — SLtt_set_color is called once. Adding the second call (with +MOST_EMBEDDED_COLOR_OFFSET) helps — colors specified in .mostrc are works as expected.

Comment 1 Adrian Reber 2015-09-03 16:42:46 UTC
Thanks for the bug report and the patch. Applied.

Comment 2 Fedora Update System 2015-09-03 16:46:07 UTC
most-5.0.0-12.a.1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-15016

Comment 3 Fedora Update System 2015-09-03 17:03:59 UTC
most-5.0.0-12.a.1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-15018

Comment 4 Fedora Update System 2015-09-04 07:29:27 UTC
most-5.0.0-12.a.1.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update most'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-15018

Comment 5 Fedora Update System 2015-09-04 07:33:32 UTC
most-5.0.0-12.a.1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update most'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-15016

Comment 6 Van de Bugger 2015-09-08 21:31:25 UTC
most-5.0.0-12.a.1.fc22 works for me.

Comment 7 Fedora Update System 2015-12-01 17:53:03 UTC
most-5.0.0-12.a.1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2015-12-01 19:50:58 UTC
most-5.0.0-12.a.1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.