Issue with the prepare_install_2_6.sh script installLmSensors function.

ggreen88 commented

2013-01-07 18:33:21 +01:00

(Migrated from github.com)

The installLmSensors function doesn't take into account the graphics card. The file it download is setup for nvidia not ati. I ended up using the HOW-TO from here http://youresuchageek.blogspot.fr/2012/06/xbmc-install-and-config-howto-for-linux.html to update it.

bramvanoploo commented

2013-01-07 18:36:19 +01:00

(Migrated from github.com)

I'm aware it's only working for Nvidia cards. I just haven't come around to adding ATI support. Thanks for scharing the howto. I'll integrate support for this in the next release.

un1versal commented

2013-02-12 13:00:56 +01:00

(Migrated from github.com)

There is zero need for some random how-to what ATI cards need is compatible advancedsettings.xml.

<advancedsettings>
  <cputempcommand>sensors|sed -ne "s/temp1: \+[-+]\([0-9]\+\).*/\1 C/p"</cputempcommand>
  <gputempcommand>/usr/bin/aticonfig --od-gettemperature | grep Temperature | cut -f 2 -d "-" | cut -f 1 -d "." | sed -e "s, ,," | sed 's/$/ C/'</gputempcommand>
</advancedsettings>

Problem solved, not that I use ATI that was ripped from http://forum.xbmc.org/showthread.php?tid=116996

There is zero need for some random how-to what ATI cards need is compatible advancedsettings.xml. ``` <advancedsettings> <cputempcommand>sensors|sed -ne "s/temp1: \+[-+]$[0-9]\+$.*/\1 C/p"</cputempcommand> <gputempcommand>/usr/bin/aticonfig --od-gettemperature | grep Temperature | cut -f 2 -d "-" | cut -f 1 -d "." | sed -e "s, ,," | sed 's/$/ C/'</gputempcommand> </advancedsettings> ``` Problem solved, not that I use ATI that was ripped from http://forum.xbmc.org/showthread.php?tid=116996

un1versal commented

2013-02-21 01:30:13 +01:00

(Migrated from github.com)

part of this could be implemented now

96b7bcd7b1

part of this could be implemented now https://github.com/uNiversaI/xbmc-ubuntu-minimal/commit/96b7bcd7b1ff9e18b2d939385653dbca363196a9

un1versal commented

2013-02-21 01:38:45 +01:00

(Migrated from github.com)

Can this be done?

function installLmSensors()
{
    showInfo "Installing temperature monitoring package (apply all defaults)..."
    aptInstall lm-sensors
    clear
    echo ""
    echo "$(tput setaf 2)$(tput bold)INSTALLATION INFO: Please confirm all questions with ENTER (applying the suggested option)."
    echo "$(tput setaf 2)The XBMC installation will continue automatically when finished.$(tput sgr0)"
    echo ""
    echo ""

    sudo sensors-detect

    if [ ! -e "$XBMC_ADVANCEDSETTINGS_FILE" ]; then
    if [[ $GFX_CARD == NVIDIA ]]; then
        createDirectory "$TEMP_DIRECTORY" 1 0
        download $DOWNLOAD_URL"temperature_monitoring.xml"
        createDirectory "$XBMC_USERDATA_DIR" 0 0
        IS_MOVED=$(move $TEMP_DIRECTORY"temperature_monitoring.xml" "$XBMC_ADVANCEDSETTINGS_FILE")

        if [ "$IS_MOVED" == "1" ]; then
            showInfo "Temperature monitoring successfully enabled in XBMC"
        else
            showError "Temperature monitoring could not be enabled in XBMC"
        fi
    fi
fi
    if [[ $GFX_CARD == AMD ]]; then
        createDirectory "$TEMP_DIRECTORY" 1 0
        download $DOWNLOAD_URL"amd-temperature_monitoring.xml"
        createDirectory "$XBMC_USERDATA_DIR" 0 0
        IS_MOVED=$(move $TEMP_DIRECTORY"temperature_monitoring.xml" "$XBMC_ADVANCEDSETTINGS_FILE")

        if [ "$IS_MOVED" == "1" ]; then
            showInfo "Temperature monitoring successfully enabled in XBMC"
        else
            showError "Temperature monitoring could not be enabled in XBMC"
        fi
    fi

    showInfo "Temperature monitoring successfully configured"

Can this be done? ``` function installLmSensors() { showInfo "Installing temperature monitoring package (apply all defaults)..." aptInstall lm-sensors clear echo "" echo "$(tput setaf 2)$(tput bold)INSTALLATION INFO: Please confirm all questions with ENTER (applying the suggested option)." echo "$(tput setaf 2)The XBMC installation will continue automatically when finished.$(tput sgr0)" echo "" echo "" sudo sensors-detect if [ ! -e "$XBMC_ADVANCEDSETTINGS_FILE" ]; then if [[ $GFX_CARD == NVIDIA ]]; then createDirectory "$TEMP_DIRECTORY" 1 0 download $DOWNLOAD_URL"temperature_monitoring.xml" createDirectory "$XBMC_USERDATA_DIR" 0 0 IS_MOVED=$(move $TEMP_DIRECTORY"temperature_monitoring.xml" "$XBMC_ADVANCEDSETTINGS_FILE") if [ "$IS_MOVED" == "1" ]; then showInfo "Temperature monitoring successfully enabled in XBMC" else showError "Temperature monitoring could not be enabled in XBMC" fi fi fi if [[ $GFX_CARD == AMD ]]; then createDirectory "$TEMP_DIRECTORY" 1 0 download $DOWNLOAD_URL"amd-temperature_monitoring.xml" createDirectory "$XBMC_USERDATA_DIR" 0 0 IS_MOVED=$(move $TEMP_DIRECTORY"temperature_monitoring.xml" "$XBMC_ADVANCEDSETTINGS_FILE") if [ "$IS_MOVED" == "1" ]; then showInfo "Temperature monitoring successfully enabled in XBMC" else showError "Temperature monitoring could not be enabled in XBMC" fi fi showInfo "Temperature monitoring successfully configured" ```

bramvanoploo commented

2013-02-21 11:22:50 +01:00

(Migrated from github.com)

I'll have a look at it.

Rows
Columns

Issue with the prepare_install_2_6.sh script installLmSensors function. #2