Works great! LGTM! =)

Service fails since 2.5.12-1

Aug 08 15:02:09 box systemd[1]: Starting thermald.service - Thermal Daemon Service... ░░ Subject: A start job for unit thermald.service has begun execution ░░ Defined-By: systemd ░░ Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel ░░ ░░ A start job for unit thermald.service has begun execution. ░░ ░░ The job identifier is 2652. Aug 08 15:02:09 box thermald[1658]: Non mobile platform, exiting.. Aug 08 15:02:09 box thermald[1658]: Non mobile platform, exiting.. Aug 08 15:02:09 box thermald[1658]: THD engine init failed Aug 08 15:02:09 box systemd[1]: thermald.service: Main process exited, code=exited, status=1/FAILURE ░░ Subject: Unit process exited ░░ Defined-By: systemd ░░ Support: https://lists.freedesktop.org/mailman/listinfo/systemd-devel ░░ ░░ An ExecStart= process belonging to unit thermald.service has exited. ░░ ░░ The process' exit code is 'exited' and its exit status is 1.

I think this is the source of the problem:

## START: Set by rpmautospec
## (rpmautospec version 0.8.4)
## RPMAUTOSPEC: autorelease, autochangelog
%define autorelease(e:s:pb:n) %{?-p:0.}%{lua:
    release_number = 1;
    base_release_number = tonumber(rpm.expand("%{?-b*}%{!?-b:1}"));
    print(release_number + base_release_number - 1);
}%{?-e:.%{-e*}}%{?-s:.%{-s*}}%{!?-n:%{?dist}}
## END: Set by rpmautospec

%global pkgname thermal_daemon

Name:       thermald
Version:    2.5.12
Release:    %autorelease
Summary:    Thermal Management daemon

# Automatically converted from old format: GPLv2+ - review is highly recommended.
License:    GPL-2.0-or-later
URL:        https://github.com/intel/%{pkgname}
Source0:    %{url}/archive/v%{version}/%{name}-%{version}.tar.gz

The thermald.spec in the source rpm appears okay but someting is wrong with it, or wrong with the system algorithm. If you check the src file thd_engine.cpp it still has the fatal non-mobile error in the code, that was fixed in this pre-release commit:

https://github.com/intel/thermal_daemon/commit/9c4812f1d12a2b0b0ee286935c9b8be781728804

You can also type thermald --version at the terminal and it shows rc1

Hmm. Upstream haven't bumped the the version number:

configure.ac is still at rc1

m4_define([td_major_version], [2])
m4_define([td_minor_version], [5.12-rc1])

hence the thermald --version confusion ....

Upstream moving to 2.5.13 and including the fixes might be a good move