Page MenuHomeSolus

Fix paths
ClosedPublic

Authored by FriesischScott on Dec 18 2017, 1:44 AM.
Tags
None
Referenced Files
F11005072: D1686.diff
Tue, Aug 1, 5:24 AM
F10988982: D1686.diff
Mon, Jul 24, 1:12 PM
F10842531: D1686.id4086.diff
Jun 10 2023, 12:45 PM
F10841985: D1686.id4085.diff
Jun 10 2023, 9:43 AM
F10841481: D1686.id4087.diff
Jun 10 2023, 7:25 AM
F10834464: D1686.diff
Jun 7 2023, 10:20 AM
F10777711: D1686.id4087.diff
May 24 2023, 12:07 AM
F10777673: D1686.id4087.diff
May 23 2023, 11:54 PM
Subscribers

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
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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)
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.