Differential D9384

Use runtime install prefix instead of compile time install prefix.
AbandonedPublic
Actions

Authored by habacker on Dec 17 2017, 7:51 PM.

Details

Reviewers
ltoscano
bcooksley
dfaure
Summary

Recent code uses compile install prefix which does not apply on Windows

Test Plan

cross compiled and did run on a native system

Diff Detail

Repository
R303 KInit
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
habacker created this revision.Dec 17 2017, 7:51 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 17 2017, 7:51 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
habacker requested review of this revision.Dec 17 2017, 7:51 PM
habacker added reviewers: ltoscano, bcooksley, dfaure.Dec 17 2017, 7:52 PM
habacker edited the summary of this revision. (Show Details)
habacker edited the test plan for this revision. (Show Details)
bcooksley added a comment.Dec 17 2017, 10:10 PM

I'll leave this to someone with more experience with the kdeinit codebase to approve, but agree that this does need to be made relocatable.
This patch will probably fix a whole pile of tests on CI.

habacker abandoned this revision.Dec 18 2017, 9:07 AM

duplicate of D7708

Revision Contents
Changeset List

PathPackages
Msrc/kdeinit/kinit_win.cpp (15 lines)

Diff 24041

View Options

src/kdeinit/kinit_win.cpp

Loading...