Author Topic: Added new thermostat device  (Read 7899 times)

WhateverFits

  • Guru
  • ****
  • Posts: 230
    • View Profile
    • Sean Walker
Added new thermostat device
« on: April 09, 2013, 03:12:58 am »
Hi, I added a new thermostat device as ticket #1751 and my SqlCVS checkin failed again (seems to be the norm on this box). I just checked in a moment ago and I have no checkin numbers. Could somebody please apply my checkin? Thanks!

phenigma

  • LinuxMCE God
  • ****
  • Posts: 1758
    • View Profile
Re: Added new thermostat device
« Reply #1 on: April 09, 2013, 04:31:31 pm »
Hi, I added a new thermostat device as ticket #1751 and my SqlCVS checkin failed again (seems to be the norm on this box). I just checked in a moment ago and I have no checkin numbers. Could somebody please apply my checkin? Thanks!

Hi, thanks for creating a template!

I'll see if I can help in any way.

Did you do an sqlCVS upgrade before submitting?
Did you use sqlCVS diff or sqlCVS checkin?

The batch that did arrive on schema is incomplete.  The primary device template is there but there is no codegroup/gsd codes included in the submission.
Could you do an sqlCVS diff and see if there are remaining entries to submit?

J.

Marie.O

  • Administrator
  • LinuxMCE God
  • *****
  • Posts: 3676
  • Wastes Life On LinuxMCE Since 2007
    • View Profile
    • My Home
Re: Added new thermostat device
« Reply #2 on: April 09, 2013, 05:18:32 pm »
Just FYI: GSD codes etc are in the IR repo, and not the DCE repo

WhateverFits

  • Guru
  • ****
  • Posts: 230
    • View Profile
    • Sean Walker
Re: Added new thermostat device
« Reply #3 on: April 09, 2013, 06:47:30 pm »
I ran it both ways and SqlCVS checkin failed before it would even do anything then the diff worked and submitted. I checked both DCE and IR sections and saw that they were in the list. I did NOT run SqlCVS upgrade beforehand. Oops. I'm currently running the diff and I'll see what happens. Then I'll run an update and again see what happens.

First diff:
Just some cruft: Two delete DeviceTemplate_DeviceCategory_ControlledVia records.

After update, second diff:
Table: DeviceTemplate
        PK_DeviceTemplate   Description   Comments
   MOD   2196   AVR/AVC series (Ethe   

So, what tables' data are missing? I can pull it out manually and send it along. This installation is a bit wonky and I don't know why.

phenigma

  • LinuxMCE God
  • ****
  • Posts: 1758
    • View Profile
Re: Added new thermostat device
« Reply #4 on: April 09, 2013, 07:10:26 pm »
Hmm, that's some strange results pre- and post- update.  The data that is missing is (as posde stated) from the IR table: IR Command Codegroup and IR commands (which are actually GSD commands).

I you are comfortable with the console try gaining root and running:

Code: [Select]
/usr/pluto/bin/sqlCVS -H schema.linuxmce.org -R 3999 -U anonymous~anonymous -r dce update
Answer yes to prompts it presents you (if any).

Code: [Select]
/usr/pluto/bin/sqlCVS -H schema.linuxmce.org -R 3999 -U anonymous~anonymous -r ir update
Answer yes to prompts it presents you (if any).

Perform another diff of dce and ir tables.  You *should* see the added codegroup and commands for your new template in the ir table diff.  However, if sqlCVS screwed up it may believe they were submitted properly and not show the diffs.  In that case you may need to recreate the codegroup and commands, or send the info along in a txt file so someone else can re-create them.

Let us know what you find and how it goes.
« Last Edit: April 09, 2013, 10:57:21 pm by phenigma »

phenigma

  • LinuxMCE God
  • ****
  • Posts: 1758
    • View Profile
Re: Added new thermostat device
« Reply #5 on: April 09, 2013, 10:57:51 pm »
And so...  I am, of course, on glue.  :P   Damn lack of sleep can affect you after a while.

I have checked the IR tables, as posde reminds...   And it looks like it is all there.  I will look at it again, and approve it.   8)

Sorry for the confusion.

J.

WhateverFits

  • Guru
  • ****
  • Posts: 230
    • View Profile
    • Sean Walker
Re: Added new thermostat device
« Reply #6 on: April 09, 2013, 11:36:54 pm »
You should try switching to scented glue sticks! ;D

phenigma

  • LinuxMCE God
  • ****
  • Posts: 1758
    • View Profile
Re: Added new thermostat device
« Reply #7 on: April 10, 2013, 12:13:13 am »
You should try switching to scented glue sticks! ;D

Mmm, good idea.   ;)  So, I've approved the template, you should do an sqlCVS update and check the template.

J.
« Last Edit: April 10, 2013, 12:14:53 am by phenigma »