Firmwares 4.39.23027 and 4.38.23038

[JSWolf]

Originally Posted by DNSB

I did some testing and the only patch that failed in the ones included in the .zip file posted by JSWolf was the PDF Pan/Scan widget.

I changed the offset for the Don't uppercase header/footer text and did some cleanup in the kobopatch.yaml while running my tests such as grouping patch groups to make it easier to prevent enabling more than one group member. I also added my preferred delete the log file before starting in kobopatch.bat which makes it easier to note errors.

The log gets overwritten if it exists. So you don't need to delete it when you run the patches.

[DNSB]

Originally Posted by JSWolf

The log gets overwritten if it exists. So you don't need to delete it when you run the patches.

Last time I ran a series of tests which was a while back, the logs were appended so I added that to the batch file.

[hatuluang]

Originally Posted by JSWolf

That one didn't get the offset updated. The one that did was the dual patch to Not uppercase and change the page number format.

Anyway, I've updated the single patch and posted a new ZIP file. So if anyone reading this has downloaded the patches, please download again.

Both patches use the same offset for "Don't uppercase header/footer text"

[JSWolf]

Originally Posted by DNSB

I did some testing and the only patch that failed in the ones included in the .zip file posted by PSWolf was the PDF Pan/Scan widget.

I changed the offset for the Don't uppercase header/footer text and did some cleanup in the kobopatch.yaml while running my tests such as grouping patch groups to make it easier to prevent enabling more than one group member. I also added my preferred delete the log file before starting in kobopatch.bat which makes it easier to note errors.

I like your grouping.

You've put a space after the groups. A space above would make it easier to read.

[JSWolf]

Originally Posted by DNSB

Last time I ran a series of tests which was a while back, the logs were appended so I added that to the batch file.

I just ran a test and the log file does not get appended. It gets overwritten.

[JSWolf]

Originally Posted by hatuluang

Both patches use the same offset for "Don't uppercase header/footer text"

I don't use the single patch. I use the dual patch. The single patch got left out. All the others were fixed.

Thanks for the offset for this so it could be fixed.

[DNSB]

Originally Posted by JSWolf

I like your grouping.

You've put a space after the groups. A space above would make it easier to read.

Feel free to edit the .yaml file how you want. A space above would probably make it a bit easier but I used that format for grouping since I first moved to using overrides and am used to it. The only pain are the two two-part patches split between librmsdk and libnickel.

[JSWolf]

Originally Posted by DNSB

Feel free to edit the .yaml file how you want. A space above would probably make it a bit easier but I used that format for grouping since I first moved to using overrides and am used to it. The only pain are the two two-part patches split between librmsdk and libnickel.

I originally used the yaml file in the src directories and did all the yes/no in those. It is a lot easier to do it in the main yaml.

The other thing I do is I use the main yaml to always load the better hyphenation dictionary in English.