Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 830233 - www-misc/monitorix-3.13.1: missing stylesheets from installation
Summary: www-misc/monitorix-3.13.1: missing stylesheets from installation
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Pedro
URL: https://www.monitorix.org/changelog.html
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2021-12-29 14:38 UTC by Jyrki Launonen
Modified: 2022-01-07 09:53 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Fixed ebuild (monitorix-3.13.1.ebuild,2.29 KB, text/plain)
2021-12-29 14:39 UTC, Jyrki Launonen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jyrki Launonen 2021-12-29 14:38:09 UTC
monitorix since 3.13.0 has theming support that uses css files, controllable via theme_color configuration variable. Those files are not installed to the system resulting somewhat broken pages.

Reproducible: Always

Steps to Reproduce:
(roughly)
1. emerge =www-misc/monitorix-3.13.1
2. rc-service monitorix start
3. xdg-open http://127.0.0.1:8080/monitorix

Actual Results:  
Page is displayed in browser default colors.

Expected Results:  
Page is displayed using monitorix style configuration (default: dark/black theme)

Fixed by altering ebuild slightly:

--- /usr/portage/www-misc/monitorix/monitorix-3.13.1.ebuild	2021-12-06 13:39:56.000000000 +0200
+++ /usr/portage/www-misc/monitorix/monitorix-3.13.1.ebuild	2021-12-06 13:39:56.000000000 +0200
@@ -60,6 +60,9 @@
 	insinto /var/lib/${PN}/www
 	doins logo_bot.png logo_top.png ${PN}ico.png
 
+	insinto /var/lib/${PN}/www/css
+	doins css/white.css css/black.css
+
 	keepdir /var/lib/${PN}/www/imgs
 	fowners monitorix:monitorix /var/lib/${PN}/www/imgs
Comment 1 Jyrki Launonen 2021-12-29 14:39:32 UTC
Created attachment 760752 [details]
Fixed ebuild
Comment 2 Pedro 2021-12-30 11:00:14 UTC
I've created the PR https://github.com/gentoo/gentoo/pull/23581 to fix this issue. Thanks for reporting it.
Comment 3 Larry the Git Cow gentoo-dev 2022-01-07 05:16:51 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=36e1ecce4533635b0b911a3fa28d6bcb59ae3596

commit 36e1ecce4533635b0b911a3fa28d6bcb59ae3596
Author:     Pedro Arizmendi <dwosky@pm.me>
AuthorDate: 2021-12-30 10:15:40 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-01-07 05:16:33 +0000

    www-misc/monitorix: Added missing css styles
    
    Signed-off-by: Pedro Arizmendi <dwosky@pm.me>
    Bug: https://bugs.gentoo.org/830233
    Package-Manager: Portage-3.0.28, Repoman-3.0.3
    Closes: https://github.com/gentoo/gentoo/pull/23581
    Signed-off-by: Sam James <sam@gentoo.org>

 www-misc/monitorix/monitorix-3.13.1-r1.ebuild | 93 +++++++++++++++++++++++++++
 1 file changed, 93 insertions(+)