Slavek,
Tim has 2 commits that implement support for Cups 1.6 we need to test and backport to 3513-sru. They are:
9bc0d2cd9d 91bf63b43b
I will try to patch a local copy and build with it, but if you can look at them as well, that be appreciated. I'll report back after my test run.
On 08/12/2012 11:12 PM, David C. Rankin wrote:
Slavek,
Tim has 2 commits that implement support for Cups 1.6 we need to test and backport to 3513-sru. They are:
9bc0d2cd9d 91bf63b43b
I will try to patch a local copy and build with it, but if you can look at them as well, that be appreciated. I'll report back after my test run.
It tdelibs builds fine in 3513-sru with the cups 1.6 commits applied (for cups < 1.6). You need to run:
sed -i 's/tde/kde/g' <commit.diff>
on each of the diff files before applying them (cherry picking them). This is due to the change from tdelibs/kdeprint to tdelibs/tdeprint and one instance of tdeprint_printer that went from kdeprint->tdeprint. This will fix the following name changes in the commits:
23:54 alchemy:~/tde/tmp> grep tde tdelibs-cups-1344635485:9bc0d2cd9d38750658770e69bf0445dc5162beb7.diff diff --git a/tdeprint/cups/cupsdconf2/cupsdconf.cpp b/tdeprint/cups/cupsdconf2/cupsdconf.cpp --- a/tdeprint/cups/cupsdconf2/cupsdconf.cpp +++ b/tdeprint/cups/cupsdconf2/cupsdconf.cpp diff --git a/tdeprint/cups/ipprequest.cpp b/tdeprint/cups/ipprequest.cpp --- a/tdeprint/cups/ipprequest.cpp +++ b/tdeprint/cups/ipprequest.cpp diff --git a/tdeprint/cups/ipprequest.h b/tdeprint/cups/ipprequest.h --- a/tdeprint/cups/ipprequest.h +++ b/tdeprint/cups/ipprequest.h diff --git a/tdeprint/cups/kmcupsjobmanager.cpp b/tdeprint/cups/kmcupsjobmanager.cpp --- a/tdeprint/cups/kmcupsjobmanager.cpp +++ b/tdeprint/cups/kmcupsjobmanager.cpp diff --git a/tdeprint/cups/kmcupsmanager.cpp b/tdeprint/cups/kmcupsmanager.cpp --- a/tdeprint/cups/kmcupsmanager.cpp +++ b/tdeprint/cups/kmcupsmanager.cpp diff --git a/tdeprint/cups/kmcupsuimanager.cpp b/tdeprint/cups/kmcupsuimanager.cpp --- a/tdeprint/cups/kmcupsuimanager.cpp +++ b/tdeprint/cups/kmcupsuimanager.cpp diff --git a/tdeprint/cups/kmwfax.cpp b/tdeprint/cups/kmwfax.cpp --- a/tdeprint/cups/kmwfax.cpp +++ b/tdeprint/cups/kmwfax.cpp diff --git a/tdeprint/cups/kmwippselect.cpp b/tdeprint/cups/kmwippselect.cpp --- a/tdeprint/cups/kmwippselect.cpp +++ b/tdeprint/cups/kmwippselect.cpp + m_list->insertItem(SmallIcon("tdeprint_printer"),TQString::fromLatin1(ippGetString(attr, 0, NULL)));
m_list->insertItem(SmallIcon("tdeprint_printer"),TQString::fromLatin1(attr->values[0].string.text)); 23:54 alchemy:~/tde/tmp> grep tde tdelibs-cups_1344730095:91bf63b43bf4cc9ff640bd3c11549644cef05e6e.diff diff --git a/tdeprint/cups/ipprequest.h b/tdeprint/cups/ipprequest.h --- a/tdeprint/cups/ipprequest.h +++ b/tdeprint/cups/ipprequest.h
** this is where needless renaming kde->tde hurts....
I will update the build environment and test with cups > 1.6 in a day or two, but I've tested and it is safe for building with cups < 1.6.
On 08/12/2012 11:58 PM, David C. Rankin wrote:
It tdelibs builds fine in 3513-sru with the cups 1.6 commits applied (for cups < 1.6). You need to run:
sed -i 's/tde/kde/g' <commit.diff>
Slavek,
Here is an updated patch for 3513-sru with the names already changed. In the R14 commits, the first hunk is off by 5 lines in context. This corrects that issue as well as the tde->kde name changes... If nothing else, this will provide the correct lines and changes.
On 08/13/2012 09:20 AM, David C. Rankin wrote:
Slavek,
Here is an updated patch for 3513-sru with the names already changed. In the R14 commits, the first hunk is off by 5 lines in context. This corrects that issue as well as the tde->kde name changes... If nothing else, this will provide the correct lines and changes.
Slavek,
I have tested this patch with both cups >= 1.6 and cups <= 1.6 and tdelibs builds fine. I think we can add it to 3513-sru. Please review the original commits 9bc0d2cd9d, 91bf63b43b (noting the need to "sed -i 's/tde/kde/g' <commit.diff>"), and review the patch I attached earlier in this thread, and push to GIT if it is OK.
Me: signoff for both cups >= 1.6 and cups <= 1.6
Dne út 14. srpna 2012 David C. Rankin napsal(a):
On 08/13/2012 09:20 AM, David C. Rankin wrote:
Slavek,
Here is an updated patch for 3513-sru with the names already changed. In the R14 commits, the first hunk is off by 5 lines in context. This corrects that issue as well as the tde->kde name changes... If nothing else, this will provide the correct lines and changes.
Slavek,
I have tested this patch with both cups >= 1.6 and cups <= 1.6 and tdelibs builds fine. I think we can add it to 3513-sru. Please review the original commits 9bc0d2cd9d, 91bf63b43b (noting the need to "sed -i 's/tde/kde/g' <commit.diff>"), and review the patch I attached earlier in this thread, and push to GIT if it is OK.
Me: signoff for both cups >= 1.6 and cups <= 1.6
David,
thank you for testing. Patch for v3.5.13-sru you adjust properly. Just a small note: I will use patches so divided, as were the original commits - to maintain information about the original git hashes, which was made of cherry-pick.
Slavek --
On 08/14/2012 10:11 AM, Slávek Banko wrote:
David,
thank you for testing. Patch for v3.5.13-sru you adjust properly. Just a small note: I will use patches so divided, as were the original commits - to maintain information about the original git hashes, which was made of cherry-pick.
Slavek
I think that is the best idea. I didn't know if 'cherry-pick' would work in this case since there was a 'tde'/'kde' difference in the commits as well as there being a 5 line context difference in the patch file. (I just don't know how cherry-pick works) If you can cherry-pick it, that is much better than another patch.
Let me know when they are available and I will test the tdelibs build again.
On Tuesday 14 of August 2012 21:56:57 David C. Rankin wrote:
On 08/14/2012 10:11 AM, Slávek Banko wrote:
David,
thank you for testing. Patch for v3.5.13-sru you adjust properly. Just a small note: I will use patches so divided, as were the original commits - to maintain information about the original git hashes, which was made of cherry-pick.
Slavek
I think that is the best idea. I didn't know if 'cherry-pick' would work in this case since there was a 'tde'/'kde' difference in the commits as well as there being a 5 line context difference in the patch file. (I just don't know how cherry-pick works) If you can cherry-pick it, that is much better than another patch.
Let me know when they are available and I will test the tdelibs build again.
As you correctly assume, git cherry-pick would fail due to renaming of file names and also texts within files. Therefore, even I not use directly git cherry-pick, but using the git show to save a patch, make the necessary adjustments and then make a commit as it was a cherry-pick.
Slavek --
On Monday 13 of August 2012 16:20:16 David C. Rankin wrote:
On 08/12/2012 11:58 PM, David C. Rankin wrote:
It tdelibs builds fine in 3513-sru with the cups 1.6 commits applied (for cups < 1.6). You need to run:
sed -i 's/tde/kde/g' <commit.diff>
Slavek,
Here is an updated patch for 3513-sru with the names already changed. In the R14 commits, the first hunk is off by 5 lines in context. This corrects that issue as well as the tde->kde name changes... If nothing else, this will provide the correct lines and changes.
Pushed to v3.5.13-sru.
Slavek --