Re: [concordance-devel] Preparing for 0.20's release
Phil Dibowitz wrote:
Stephen Warren wrote:
There is one cosmetic bug, in that print CTRL-H doesn't do a backspace
on Windows, so the percentage progress messages looks like turd. I'll
see if that's easy to fix...
Eh, what? That should never happen on windows:
#ifdef WIN32
GetConsoleScreenBufferInfo(con, &sbi);
SetConsoleCursorPosition(con, sbi.dwCursorPosition);
#else
if (count != 0) {
printf("\b\b\b\b\b\b\b\b\b\b\b\b\b\b");
}
#endif
That code never gets executed if WIN32 is defined... something's very odd...
Yeah, I hadn't looked at the code, and was assuming that ^H wasn't
working. In fact, it the issue was that the WIN32 code branch is a no-op.
Attached is a patch that:
concordance.c:
Fix progress message formatting on WIN32 by actually adjusting
the cursor position.
Fix some trailing white-space.
Localize a variable declaration to where it's used.
Note: This is another Windows/Cygwin-based patch, so if it fails to
apply, maybe utod is required.
? concordance/win/Debug
? concordance/win/Release
? concordance/win/concordance.vcproj.ESK.Stephen Warren.user
? consnoop/win/consnoop.vcproj.ESK.Stephen Warren.user
? libconcord/win/Debug
? libconcord/win/Release
? libconcord/win/libconcord_libusb.vcproj.ESK.Stephen Warren.user
? libconcord/win/libconcord_winhid.vcproj.ESK.Stephen Warren.user
? win/concordance.ncb
? win/concordance.suo
Index: concordance/concordance.c
===
RCS file: /cvsroot/concordance/concordance/concordance/concordance.c,v
retrieving revision 1.24
diff -u -p -r1.24 concordance.c
--- concordance/concordance.c 11 Apr 2008 05:05:26 - 1.24
+++ concordance/concordance.c 11 Apr 2008 05:50:18 -
@@ -50,7 +50,6 @@ char* basename(char* file_name)
}
HANDLE con;
-CONSOLE_SCREEN_BUFFER_INFO sbi;
#else
/* NON-Windows */
@@ -106,13 +105,22 @@ void cb_print_percent_status(uint32_t co
{
int is_bytes;
#ifdef WIN32
- GetConsoleScreenBufferInfo(con, &sbi);
- SetConsoleCursorPosition(con, sbi.dwCursorPosition);
-#else
+ CONSOLE_SCREEN_BUFFER_INFO sbi;
+#endif
+
if (count != 0) {
+#ifdef WIN32
+ GetConsoleScreenBufferInfo(con, &sbi);
+ sbi.dwCursorPosition.X -= 14;
+ if (sbi.dwCursorPosition.X < 0) {
+ sbi.dwCursorPosition.X = 0;
+ }
+ SetConsoleCursorPosition(con, sbi.dwCursorPosition);
+#else
printf("\b\b\b\b\b\b\b\b\b\b\b\b\b\b");
+#endif
}
-#endif
+
is_bytes = 0;
if (arg) {
is_bytes = (int)(arg);
@@ -123,7 +131,7 @@ void cb_print_percent_status(uint32_t co
} else {
printf("%3i%% ", curr*100/total);
}
- fflush(stdout);
+ fflush(stdout);
}
/*
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Preparing for 0.20's release
Stephen Warren wrote:
> Phil Dibowitz wrote:
>> - final test on Windows of current code
>>
>> This hasn't been done - could someone do this for me?
>
> I tested all these on Windows, built with Visual C++ 2005 Express
> Edition, against my 880:
>
> Debug: Compile -c -C -f -F -h -i -k -K -s -t -v
> Release: Compile
>
> There is one cosmetic bug, in that print CTRL-H doesn't do a backspace
> on Windows, so the percentage progress messages looks like turd. I'll
> see if that's easy to fix...
Eh, what? That should never happen on windows:
#ifdef WIN32
GetConsoleScreenBufferInfo(con, &sbi);
SetConsoleCursorPosition(con, sbi.dwCursorPosition);
#else
if (count != 0) {
printf("\b\b\b\b\b\b\b\b\b\b\b\b\b\b");
}
#endif
That code never gets executed if WIN32 is defined... something's very odd...
--
Phil Dibowitz [EMAIL PROTECTED]
Open Source software and tech docsInsanity Palace of Metallica
http://www.phildev.net/ http://www.ipom.com/
"Never write it in C if you can do it in 'awk';
Never do it in 'awk' if 'sed' can handle it;
Never use 'sed' when 'tr' can do the job;
Never invoke 'tr' when 'cat' is sufficient;
Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming
signature.asc
Description: OpenPGP digital signature
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Preparing for 0.20's release
Phil Dibowitz wrote: > - final test on Windows of current code > > This hasn't been done - could someone do this for me? I tested all these on Windows, built with Visual C++ 2005 Express Edition, against my 880: Debug: Compile -c -C -f -F -h -i -k -K -s -t -v Release: Compile There is one cosmetic bug, in that print CTRL-H doesn't do a backspace on Windows, so the percentage progress messages looks like turd. I'll see if that's easy to fix... - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Preparing for 0.20's release
Phil Dibowitz wrote: > Anyone else have anything they think needs to be in 0.20? Well, since I've been using Doxygen a lot at work, I was contemplating writing a bunch of Doxygen comments in libconcord.h, so we could generate pretty looking doc pages etc., now that the API seems reasonably stable for a while. That said, I certainly wouldn't actually hold up any given release for this. - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Preparing for 0.20's release
Phil Dibowitz wrote: > Hey folks, > > Despite the firmware upgrade code not working on all remotes yet, I'd like > to start prep'ing for the 0.20 release and aim to get it out early next > week. The rename and library split are stable at this point, and the > majority of the API is stable. > > Here's what I see as needing to be done before release: > > - rename libconcord/concordance.h to something else (see previous email) > - I'd like to get the setupapi.h thing worked out of possible > - ask SF to remove the whole examples/ directory (I'll do that today) These three are done (SF hadn't moved the examples/ directory yet, but the request is in). > - final test on Windows of current code This hasn't been done - could someone do this for me? I'm planning to release this weekend or early next week unless I hear otherwise. One thing I was thinking about - instead of allowing people to do firmware upgrades on un-proven platforms (arch's 3 and 9), even though it shouldn't cause any harm, I think it's quite confusing when it doesn't work. I'm thinking of disabling that ability for the release, and then re-enabling it in CVS so we can continue to work on it. Thoughts? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Marco: setupapi.h question
Phil Dibowitz wrote: >> By my understanding, development on Windows pretty much always requires >> both a C compiler and a platform SDK installation, so this shouldn't be >> anything new to people building apps from source on Windows. >> >> So, it seems we can remove setupapi.h from CVS, and add notes to >> INSTALL.windows that the platform SDK is required. >> >> Marco: Do you agree? > > Marco, any word on this? OK, well, I went ahead and nuked this and added a note to the INSTALL.windows. Let me know if there are any problems. Everyone's been really quite the last few days - you all still alive? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Dealing with libconcord/concordance.h
Phil Dibowitz wrote: > One of the things I want to cleanup before we get 0.20 out the door is > libconcord/concordance.h. There's not much left in this file, but it makes > no sense for anything called 'concordance' to be in libconcord/ Well, I got no feedback here, so I'm renaming it to lc_internal.h -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
On Thursday 10 April 2008, Phil Dibowitz wrote: > Thanks. That's a whole lot of work when you coulda just opened firefox in > linux, selected the keys you wanted, downloaded the XML file from the site > and sent it along rather than piecing it together with wireshark. :) Well, I was just curious to look at the internals of the original software at work with learning IR codes, so I sniffed net and USB in parallel, and the XML file fell out of the whole bunch among others - though I still haven't found enough time for a deeper look into the details. And, yes, you can learn multiple keys in a batch with the Logitech software. The Javascript will put the next key's name in a listbox on the learning page, check it when it has been learned and add the next key to the list. Andreas - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
On Thu, Apr 10, 2008 at 11:13 PM, Phil Dibowitz <[EMAIL PROTECTED]> wrote: > This is odd, I get great reception with my 880. Either the 5 series us > significantly less powered than the 8 series, or your batteries are very > weak. > I think it's the battery as well. > As for the one-file-per-key, does the official software allow you to learn > more than one at a time? Sadly I don't have access to a Windows machine nor a Mac, so I can't tell, really. But the webinterface, and the XML file leads me to believe that it does support it. But I do believe that it involves quite a bit of communicating to the server to tell it that you've moved on to the next key. (it keeps track of that) And for some keys, such as the Teletext key and the red key make the connection bad, at least after setting those, I had to reboot the remote in order to get it identified again. I'll try it again this weekend, and make a list of the keys that break the identification until reboot. greetings, Marcel de Jong - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
Andreas Schulz wrote: > On Thursday 10 April 2008, Marcel de Jong wrote: >> I grabbed the CVS version yesterday, and had it run, but it only took >> one command in the file. I had to download the learnIr file for each >> and every command separately. I kinda expected it to read through all >> the key commands selected in the screen, and which are available in >> the file. Bad assumption on my part? > > I'm afraid, yes - LearnIR will currently only learn the first key > from the xml file, even if you selected multiple keys. > I was just about to post a multiple-key xml file, just in case > Phil or Steven would be interested to see one - see attachment: > It's from a wireshark sniff of a session with the cuurent Windows > software, and piped through tidy to make the XML better readable. > (and anonymized to protect my privacy). Thanks. That's a whole lot of work when you coulda just opened firefox in linux, selected the keys you wanted, downloaded the XML file from the site and sent it along rather than piecing it together with wireshark. :) > As I just wrote in this thread, there is still a bug in remote.cpp, > which means that you learned nothing but garbage. Not anymore. :) -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
Marcel de Jong wrote: > It does learn it, sadly the remote still does not turn my TV on or > off, not sure why though (not a big problem, since I turn the TV off > completely anyway), and the distance between tv IR-eye and remote IR > transmitter needs to be pretty close. > This latter issue kinda renders the remote rather useless for me. :( > It has nothing to do with the work you guys perform, concordance works > for me, it's just that the remote itself seems a paperweight to me. :) This is odd, I get great reception with my 880. Either the 5 series us significantly less powered than the 8 series, or your batteries are very weak. As for the one-file-per-key, does the official software allow you to learn more than one at a time? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
Andreas Schulz wrote: > On Thursday 10 April 2008, Phil Dibowitz wrote: >> No need to initialize that err there once the if() is removed. And yes, I >> removed the if() before I sent the email. The commit message was in my >> inbox at 16:46 on 4/8. > Phil, > you're right - CVS just fooled me since I didn't update recently and > it still showed the diff of mine against the previous version. > That leaves the index bug in remote.cpp to be fixed in CVS. Ooops, I missed that. Patch applied, thanks. -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
Stephen Warren wrote:
> On Thu, April 10, 2008 1:25 pm, Phil Dibowitz wrote:
>> Michael Frase wrote:
>>> Hi Phil,
>>>
>>> thanks for your quick replies and changes in cvs. I'm back again and
>>> tested latest cvs with my harmony 555.
>> ...
>>> Here's whats REALLY odd. I massaged the LatestUpdate.EZUp to be in the
>> format we write out fw dumps in, and it's IDENTICAL (minus the magic
>> bytes, of course) to his current fw dump:
>> ...
>> Meanwhile, there's drastic differences between his dump now and his dump
>> from before.
>
> Are you sure this remote/arch has the correct firmware upgrade base? I.e
> are we writing to and dumping the wrong location?
According to the usblogs from Kevin:
Erase Flash 81
Write Flash 81 3150
...
and from remote_info.h:
//fl_base, fw_base,config_base,fw_up_base, fw_4847_off,
cookie, ck_sz, endvec, micro, fl_sz, ram_sz, ee_sz, usb
...
// 9 - 360, 52x, 55x
{ 0x80, 0x81, 0x82, 0x81, 4,
0x4D434841, 4, 4, "PIC18LF4550", 16, 2048, 256,
"Internal" },
It writes directly over the live firmware in fact.
--
Phil Dibowitz [EMAIL PROTECTED]
Open Source software and tech docsInsanity Palace of Metallica
http://www.phildev.net/ http://www.ipom.com/
"Never write it in C if you can do it in 'awk';
Never do it in 'awk' if 'sed' can handle it;
Never use 'sed' when 'tr' can do the job;
Never invoke 'tr' when 'cat' is sufficient;
Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming
signature.asc
Description: OpenPGP digital signature
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
On Thursday 10 April 2008, Marcel de Jong wrote: > I grabbed the CVS version yesterday, and had it run, but it only took > one command in the file. I had to download the learnIr file for each > and every command separately. I kinda expected it to read through all > the key commands selected in the screen, and which are available in > the file. Bad assumption on my part? I'm afraid, yes - LearnIR will currently only learn the first key from the xml file, even if you selected multiple keys. I was just about to post a multiple-key xml file, just in case Phil or Steven would be interested to see one - see attachment: It's from a wireshark sniff of a session with the cuurent Windows software, and piped through tidy to make the XML better readable. (and anonymized to protect my privacy). > It does learn it, sadly the remote still does not turn my TV on or > off, not sure why though (not a big problem, since I turn the TV off > completely anyway), and the distance between tv IR-eye and remote IR > transmitter needs to be pretty close. As I just wrote in this thread, there is still a bug in remote.cpp, which means that you learned nothing but garbage. Andreas 2.3 2.4 2.5 2.7 2.8 Warning WARNING: This version of the software is no longer supported. Please click on the 'Software' link in the top right corner of your browser to get the latest version. 2.3 2.4 2.5 2.7 2.8 Abort Winsock 1 KeyName 0 KeyName 1 KeyName 2 KeyName 3 KeyName 4 KeyName 5 MessagePressKey Press and release the '%KEYNAME%' on your original remote control. MessagePressKeyGeneral Press and release the '%KEYNAME%' on your original remote control. InstructionAcquiring As shown, start with the remote controls 2 inches (5 cm) apart, then press and release the '%KEYNAME%' button. InstructionAcquiringGeneral As shown, start with the remote controls 2 inches (5 cm) apart, then press and release the '%KEYNAME%' button. MessageKeyDetected Learning successful! MessageIrDetected Please try again. myremotesetup.com EasyZapper/New/ProcDigitizeInfrared/LearnIr_Receive.asp 3 Cookie CookieKeyValue=%7BEE- AUSER=A--; PSWD=KI--- HR%5FUSER%5FLANGUAGE=enu; SKIN%5FID=Tonic; HRLB=17--; ASPSESSIONIDQCQSCATC=MIi- UserId 25-- UserDeviceId 20-- KeyName 5 myremotesetup.com EasyZapper/New/ProcDigitizeInfrared/LearnIr_Receive_Complete.asp 3 - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
On Thursday 10 April 2008, Phil Dibowitz wrote: > No need to initialize that err there once the if() is removed. And yes, I > removed the if() before I sent the email. The commit message was in my > inbox at 16:46 on 4/8. Phil, you're right - CVS just fooled me since I didn't update recently and it still showed the diff of mine against the previous version. That leaves the index bug in remote.cpp to be fixed in CVS. Andreas - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
> > On Wednesday 09 April 2008, Phil Dibowitz wrote: > >> Marcel de Jong wrote: > >>> Hello all, > >>> > >>> I'm afraid I have some bad news, not sure if others have had this as > >>> well. When I tried to set up my 555, I needed to have the remote learn > >>> some new trick via IR (basically because it refused to turn on or off me > >>> TV and all other devices I have), but when I ran the command it gave me > >>> this error message: > >> It got broken sometime recently - when the file_read() stuff was done. I > >> fixed it in CVS, thanks for pointing that out. > > I grabbed the CVS version yesterday, and had it run, but it only took one command in the file. I had to download the learnIr file for each and every command separately. I kinda expected it to read through all the key commands selected in the screen, and which are available in the file. Bad assumption on my part? It does learn it, sadly the remote still does not turn my TV on or off, not sure why though (not a big problem, since I turn the TV off completely anyway), and the distance between tv IR-eye and remote IR transmitter needs to be pretty close. This latter issue kinda renders the remote rather useless for me. :( It has nothing to do with the work you guys perform, concordance works for me, it's just that the remote itself seems a paperweight to me. :) I'll try again this weekend when I have more time to work with the remote. Perhaps the batteries are running flat. greetings, Marcel - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
On Thu, April 10, 2008 1:25 pm, Phil Dibowitz wrote: > Michael Frase wrote: >> Hi Phil, >> >> thanks for your quick replies and changes in cvs. I'm back again and >> tested latest cvs with my harmony 555. > > ... > > Here's whats REALLY odd. I massaged the LatestUpdate.EZUp to be in the > format we write out fw dumps in, and it's IDENTICAL (minus the magic > bytes, of course) to his current fw dump: > ... > Meanwhile, there's drastic differences between his dump now and his dump > from before. Are you sure this remote/arch has the correct firmware upgrade base? I.e are we writing to and dumping the wrong location? - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
Andreas Schulz wrote: > On Wednesday 09 April 2008, Phil Dibowitz wrote: >> Marcel de Jong wrote: >>> Hello all, >>> >>> I'm afraid I have some bad news, not sure if others have had this as >>> well. When I tried to set up my 555, I needed to have the remote learn >>> some new trick via IR (basically because it refused to turn on or off me >>> TV and all other devices I have), but when I ran the command it gave me >>> this error message: >> It got broken sometime recently - when the file_read() stuff was done. I >> fixed it in CVS, thanks for pointing that out. > > Did you? Cannot see any change yet.. In case you didn't have time, I found > meanwhile that it's actually two bugs - one probably from the file_read > changes in libharmony.cpp, which fails because checking an unset err > variable, and the next one in remote.cpp from the USB buffer changes, to the > effect that the length byte now sits in rsp[63], not in rsp[64]. No need to initialize that err there once the if() is removed. And yes, I removed the if() before I sent the email. The commit message was in my inbox at 16:46 on 4/8. -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Learn-IR not yet working?
On Wednesday 09 April 2008, Phil Dibowitz wrote:
> Marcel de Jong wrote:
> > Hello all,
> >
> > I'm afraid I have some bad news, not sure if others have had this as
> > well. When I tried to set up my 555, I needed to have the remote learn
> > some new trick via IR (basically because it refused to turn on or off me
> > TV and all other devices I have), but when I ran the command it gave me
> > this error message:
>
> It got broken sometime recently - when the file_read() stuff was done. I
> fixed it in CVS, thanks for pointing that out.
Did you? Cannot see any change yet.. In case you didn't have time, I found
meanwhile that it's actually two bugs - one probably from the file_read
changes in libharmony.cpp, which fails because checking an unset err
variable, and the next one in remote.cpp from the USB buffer changes, to the
effect that the length byte now sits in rsp[63], not in rsp[64].
Attached patch should fix both.
Andreas
Index: libconcord.cpp
===
RCS file: /cvsroot/concordance/concordance/libconcord/libconcord.cpp,v
retrieving revision 1.26
diff -u -3 -p -u -p -r1.26 libconcord.cpp
--- libconcord.cpp 4 Apr 2008 09:16:08 - 1.26
+++ libconcord.cpp 10 Apr 2008 19:39:52 -
@@ -1215,12 +1215,9 @@ int extract_firmware_binary(uint8_t *xml
int learn_ir_commands(uint8_t *data, uint32_t size, int post)
{
- int err;
+ int err = 0;
if (data) {
- if (err != 0) {
- return err;
- }
uint8_t *t = data;
string keyname;
Index: remote.cpp
===
RCS file: /cvsroot/concordance/concordance/libconcord/remote.cpp,v
retrieving revision 1.30
diff -u -3 -p -u -p -r1.30 remote.cpp
--- remote.cpp 4 Apr 2008 09:16:09 - 1.30
+++ remote.cpp 10 Apr 2008 19:39:52 -
@@ -638,7 +638,7 @@ int handle_ir_response(uint8_t rsp[64],
unsigned int &t_on, unsigned int &t_off, unsigned int &pulse_count,
unsigned int *&pulses, unsigned int &freq)
{
- const unsigned int len = rsp[64];
+ const unsigned int len = rsp[63];
if ((len & 1) == 0) {
for (unsigned int u = 2; u < len; u += 2) {
const unsigned int t = rsp[u] << 8 | rsp[1+u];
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel
[concordance-devel] Preparing for 0.20's release
Hey folks, Despite the firmware upgrade code not working on all remotes yet, I'd like to start prep'ing for the 0.20 release and aim to get it out early next week. The rename and library split are stable at this point, and the majority of the API is stable. Here's what I see as needing to be done before release: - rename libconcord/concordance.h to something else (see previous email) - I'd like to get the setupapi.h thing worked out of possible - final test on Windows of current code - ask SF to remove the whole examples/ directory (I'll do that today) Anyone else have anything they think needs to be in 0.20? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Marco: setupapi.h question
Stephen Warren wrote: > Phil Dibowitz wrote: >> Stephen Warren wrote: >>> A couple of license-related questions >>> >>> 1) The file libconcord/win/setupapi.h states: >>> >>> Copyright (c) 1995-1997 Microsoft Corporation >>> >>> This certainly isn't GPL, but it's possible MS allows redistribution. >>> >>> The thing is, I'd expect any Windows C compiler (or perhaps the platform >>> SDK) to include this header, so I'm not sure why libconcord needs to >>> ship it at all? >> I'd expect that too, but Kevin did all of the Windows stuff based on some >> version of Visual C++ from like... 1942. That's why we have all kinds of >> stuff. If someone with greater knowledge of the platform than I wants to >> speak on how much this is needed I'm definitely happy to entertain nuking >> them. > > It looks like we don't need to ship setupapi.h at all. > > I deleted this file from libconcord, made sure I had the Microsoft > Platform SDK installed, opened the .sln file using MS Visual Studio 2005 > Express, adjusted the include paths to point at the platform SDK, and > everything builds just fine. > > By my understanding, development on Windows pretty much always requires > both a C compiler and a platform SDK installation, so this shouldn't be > anything new to people building apps from source on Windows. > > So, it seems we can remove setupapi.h from CVS, and add notes to > INSTALL.windows that the platform SDK is required. > > Marco: Do you agree? Marco, any word on this? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
Michael Frase wrote: > Hi Phil, > > thanks for your quick replies and changes in cvs. I'm back again and > tested latest cvs with my harmony 555. Michael sent me, off-list, a his LatestUpdate.EZUp, and a firmware dump from both before and after the upgrade. Here's whats REALLY odd. I massaged the LatestUpdate.EZUp to be in the format we write out fw dumps in, and it's IDENTICAL (minus the magic bytes, of course) to his current fw dump: - CAEF3FF012000201C86BC96B010109A537D08D940101 + 43724847CAEF3FF012000201C86BC96B010109A537D08D940101 Those are the only two lines that show up in the diff. Meanwhile, there's drastic differences between his dump now and his dump from before. In other words, it appears to be running on the latest firmware, but reporting old firmware. Honestly, I haven't even the slightest clue why... much less why Michel and Stefan can't upgrade successfully, but Marcel can, which is an even stranger mystery. It would be interesting to see a firmware dump from an upgraded unit - Marcel, can you do a dump from your remote and send it to me. I'm not sure it'll show me anything new, but I'm kinda at a loss for better ideas. -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
Am Donnerstag, den 10.04.2008, 10:44 -0700 schrieb Phil Dibowitz: > Michael Frase wrote: > > Hi Phil, > > > > thanks for your quick replies and changes in cvs. I'm back again and > > tested latest cvs with my harmony 555. > > > > Unfortunately firmware update doesn't work as expected. The update > > process itself seems to be ok... But firmware version remains the same > > (Update from v2.5 to v2.6). > > Hmm. OK, that's the second report of that. Does your remote boot into > safemode after the upgrade? Does it reboot at all? The remote didn't reboot in safemode. It rebooted and started in activity mode. I saw a "growing bar" during reboot. I tried to update several times without success. The Remote seems to work without problems after all... Is it possible that the new firmware data was written correctly, but the firmware version wasn't updated? > > - > This SF.net email is sponsored by the 2008 JavaOne(SM) Conference > Don't miss this year's exciting event. There's still time to save $100. > Use priority code J8TL2D2. > http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone > ___ concordance-devel mailing > list concordance-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/concordance-devel - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
Michael Frase wrote: > Hi Phil, > > thanks for your quick replies and changes in cvs. I'm back again and > tested latest cvs with my harmony 555. > > Unfortunately firmware update doesn't work as expected. The update > process itself seems to be ok... But firmware version remains the same > (Update from v2.5 to v2.6). Hmm. OK, that's the second report of that. Does your remote boot into safemode after the upgrade? Does it reboot at all? -- Phil Dibowitz [EMAIL PROTECTED] Open Source software and tech docsInsanity Palace of Metallica http://www.phildev.net/ http://www.ipom.com/ "Never write it in C if you can do it in 'awk'; Never do it in 'awk' if 'sed' can handle it; Never use 'sed' when 'tr' can do the job; Never invoke 'tr' when 'cat' is sufficient; Avoid using 'cat' whenever possible" -- Taylor's Laws of Programming signature.asc Description: OpenPGP digital signature - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
Re: [concordance-devel] Firmware Upgrade 525
Hi Phil, thanks for your quick replies and changes in cvs. I'm back again and tested latest cvs with my harmony 555. Unfortunately firmware update doesn't work as expected. The update process itself seems to be ok... But firmware version remains the same (Update from v2.5 to v2.6). > $ sudo concordance LatestFirmware.EZUp > Concordance 0.13+CVS > Copyright 2007 Kevin Timmerman and Phil Dibowitz > This software is distributed under the GPLv3. > > Requesting Identity: 100% done > Invalidating Flash: done > Erasing Flash: 100% done > Writing firmware:100% done > Contacting website: done > Resetting... > Success! After firmware update the remote can't be identified by concordance. > $ sudo concordance -i -v > Concordance 0.13+CVS > Copyright 2007 Kevin Timmerman and Phil Dibowitz > This software is distributed under the GPLv3. > > Requesting Identity: Error requesting identity > Failed with error 1 After that first attempt the remote communicates with concordance again. > $ sudo concordance -i -v > Concordance 0.13+CVS > Copyright 2007 Kevin Timmerman and Phil Dibowitz > This software is distributed under the GPLv3. > > Requesting Identity: 100% done > Model: Logitech Harmony 555 (Mocha Grande) > Skin: 48 > Firmware Version: 2.5 > Firmware Type: 0 > Hardware Version: 3.0 > External Flash: 512 KiB - FF:12 25F040 > Architecture: 9 > Protocol: 9 > Manufacturer: Harmony Remote 0-2.5.0 > Product: Harmony Remote 0-2.5.0 > IRL, ORL, FRL: 64, 64, 0 > USB VID: 046D > USB PID: C111 > USB Ver: 0930 > Serial Number: XXX CENSORED XXX > Config Flash Used: 17% (67 of 384 KiB) > > Success! I will send you a private mail with a firmware dump of my harmony, latest firmware (website) and usbmon logs (in addition to the output above). Greetings... Michael Am Dienstag, den 08.04.2008, 20:05 -0700 schrieb Phil Dibowitz: > Stefan Arndt wrote: > >>> OK, updated cvs. > >>> > >>> But it is still Firmware 2.1 after all... > >> It looks like you may have a 2.1 firmware file. You may want to remove it > >> and then try downloading it through the site again. For what it's worth, > >> someone else with your remote was just able to upgrade to 2.6 with the > >> latest CVS. Which isn't to say there's no problems - there might still > >> be... > >> but it's at least one success... > >> > >> -- > >> Phil Dibowitz [EMAIL PROTECTED] > > > > OK, downloaded new firmware, upgraded but still 2.1 > > Bringing this discussion back to the list... Stefan, send me files off-list, > but please keep the discussion on the list. > > Marcel and Stefan, can you each send me your LatestFirmware.EZHex off-list? > Marcel, you have a 555, right, and Stefan, you have a 525? > > Thanks. > - > This SF.net email is sponsored by the 2008 JavaOne(SM) Conference > Don't miss this year's exciting event. There's still time to save $100. > Use priority code J8TL2D2. > http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone > ___ concordance-devel mailing > list concordance-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/concordance-devel - This SF.net email is sponsored by the 2008 JavaOne(SM) Conference Don't miss this year's exciting event. There's still time to save $100. Use priority code J8TL2D2. http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone ___ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel
