D22328: refactor basic OS info to be set via config and fixate it for testing

Harald Sitter noreply at phabricator.kde.org
Mon Jul 8 15:20:17 BST 2019


sitter created this revision.
sitter added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.

REVISION SUMMARY
  the basic OS info is decided at compile time, on top of that
  influences whether lsbrelease and osrelease are even run, and also
  acts as seed value for some other members.
  this resulted in tests failing on freebsd because it would detect freebsd
  as build platform and then skip all the advanced lookup tech and having
  unexpected values.
  
  fetchOSBasicInformation is now a dangling static and used to determine
  the default constructed config value of the operating system.
  this is now also forced to read Linux for the tests.
  
  I've also changed all the QLatin1Strings in the function to QStringLiteral.
  as I recall QL1S is only faster when not turning the object into a
  full QString, so these values shouldn't be QL1S.

TEST PLAN
  tests pass on linux still (and hopefully freebsd CI too)

REPOSITORY
  R871 DrKonqi

BRANCH
  sitter/fetchosbasic

REVISION DETAIL
  https://phabricator.kde.org/D22328

AFFECTED FILES
  src/systeminformation.cpp
  src/systeminformation.h
  src/tests/systeminformationtest.cpp

To: sitter, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190708/974a3620/attachment-0001.html>


More information about the Plasma-devel mailing list