Bug 1230278

Summary: color overstrike and color underline in .mostrc have no effect
Product: [Fedora] Fedora Reporter: Van de Bugger <van.de.bugger>
Component: mostAssignee: Adrian Reber <adrian>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 22CC: adrian
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
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:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-01 17:53:05 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Fix for the problem. none

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.