Page MenuHomeSolus

Fix paths
ClosedPublic

Authored by FriesischScott on Dec 18 2017, 1:44 AM.

Details

Reviewers
DataDrake
Group Reviewers
Triage Team
Commits
R3861:305bd6852eb7: Fix paths
Summary

Fixes the udev path and properly adapts the udev rules and
service to use the correct directory.

Outstanding problems:

As seen here https://github.com/MatMoul/g810-led/blob/master/INSTALL.md, the udev rules and reboot service need profiles to load from /usr/share/g810-led/profile and /usr/share/g810-led/reboot The default files are located in /usr/share/g810-led/samples and should be copied on first install.

I don't know how to make Solus create these files only on first install but not on package upgrades to not override user changes.

Test Plan

I tested it and it works now.

Diff Detail

Repository
R3861 g810-led
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

FriesischScott created this revision.Dec 18 2017, 1:44 AM
FriesischScott edited the summary of this revision. (Show Details)Dec 18 2017, 1:48 AM
FriesischScott edited the test plan for this revision. (Show Details)
FriesischScott edited the summary of this revision. (Show Details)
FriesischScott edited the summary of this revision. (Show Details)

Fix release number

Really fix release number...

DataDrake accepted this revision.Dec 18 2017, 5:38 PM
DataDrake added a subscriber: DataDrake.

100% my fault. Nice catch.

This revision is now accepted and ready to land.Dec 18 2017, 5:38 PM
This revision was automatically updated to reflect the committed changes.