kobopatch - An improved patching system - Page 5

Terisa de morgan · 07-06-2018, 06:15 PM

Quote:

Originally Posted by geek1011

I have added support for compiling and adding translation files in v0.6.4.

Terisa de morgan: Can you test it, as I don't use custom translations myself?

Tested and translations are updated (after reading converter.sh for knowing how to add the info

). I can say for sure this because I've changed a translation (I had already patched the FW) and the new translation has appeared.

My configuration file includes to different files for nickel (GeoffR and oren64) and one translation.

Thank you very much. With this tool you allow me to skip four steps: to merge GeoffR's and oren64's patches, to configure every time which patches I want to apply, to update the script for adding translations and copy binary files to the tools directory. So a big thank you.

geek1011 · 07-06-2018, 06:35 PM

Quote:

Originally Posted by Terisa de morgan

Tested and translations are updated (after reading converter.sh for knowing how to add the info

). I can say for sure this because I've changed a translation (I had already patched the FW) and the new translation has appeared.

My configuration file includes two different files for nickel (GeoffR and oren64) and one translation.

Thank you very much. With this tool you allow me to skip four steps: to merge GeoffR's and oren64's patches, to configure every time which patches I want to apply, to update the script for adding translations and copy binary files to the tools directory. So a big thank you.

Thanks for testing and responding so fast!

And you're welcome for kobopatch

. You used it exactly how I intended it to be used (and how I use it myself).

Now for zlib, then kobopatch will be complete. It's been quite tricky so far, due to implementation differences in different libraries, but the end result will be worth it (less time required to make simple modifications, and easier to update to new firmware versions).

npr · 07-07-2018, 05:11 PM

Updated my H2O e2 to the latest firmware using this patching system. Everything went well, took a while to understand how to use the kobopatch.yaml file but expect that to save lots of time on future updates.

Thanks for this new system it looks a excellent step forward.

One additional thing I would like to be able to do is replace the hyphen.en_GB file with my own custom one during the patching process. Is this possible?

geek1011 · 07-07-2018, 05:40 PM

Quote:

Originally Posted by npr

Updated my H2O e2 to the latest firmware using this patching system. Everything went well, took a while to understand how to use the kobopatch.yaml file but expect that to save lots of time on future updates.

Do you have any suggestions for how I can make it easier for new users to understand kobopatch.yaml?

Quote:

Originally Posted by npr

Thanks for this new system it looks a excellent step forward.

You're welcome.

Quote:

Originally Posted by npr

One additional thing I would like to be able to do is replace the hyphen.en_GB file with my own custom one during the patching process. Is this possible?

Probably. I am most likely going to add the ability to replace custom files sometime soon.

npr · 07-08-2018, 08:27 AM

It's really not that difficult to understand, my main problem was entering the list of patches with the yes / no flag and the inevitable typos I made which stopped the patching process and needed correcting.

The second issue I had was when I ran the new patch using the kobopatch.yaml from the previous patch version. The patching process stopped at every missing patch even though they were flagged as "no", it then took some time to identify the missing patch and remove it from the .yaml file.

One solution would be to only list "patch name: yes" in the .yalm file

Alternatively what do you think about making missing "no" patches have a soft fail. I don't see why the patch process needs to stop when a "no" patch is missing.

oren64 · 07-08-2018, 08:42 AM

Quote:

Originally Posted by geek1011

Tips and Tricks

Separating custom patches from the included ones
To make upgrading easier, as well as using the overrides section of kobopatch.yaml, you can also put custom patches which are not included into their own file. To do this, make a new file in the src folder, for example nickel.custom.patch (for patch32lsb format), or nickel.custom.yaml (for the new format). Afterwards, add a line in the patches section of kobopatch.yaml corresponding to your new file. Here is an example of mine:

Spoiler:

It's better than what I ask for, thanks I will test it today.

geek1011 · 07-08-2018, 12:10 PM

Quote:

Originally Posted by npr

It's really not that difficult to understand, my main problem was entering the list of patches with the yes / no flag and the inevitable typos I made which stopped the patching process and needed correcting.

The second issue I had was when I ran the new patch using the kobopatch.yaml from the previous patch version. The patching process stopped at every missing patch even though they were flagged as "no", it then took some time to identify the missing patch and remove it from the .yaml file.

One solution would be to only list "patch name: yes" in the .yalm file

Alternatively what do you think about making missing "no" patches have a soft fail. I don't see why the patch process needs to stop when a "no" patch is missing.

Thanks for the feedback! For making patch selection easier, what do you think of a web interface to generate kobopatch.yaml easier?

For the overrides, you only need to list the yes ones, as all patches are disabled by default. Your last suggestion sounds good though. I'll do it for the next version.

geek1011 · 07-08-2018, 12:13 PM

Quote:

Originally Posted by oren64

It's better than what I ask for, thanks I will test it today.

No problem! And I'm currently working on zlib support.

Semwize · 07-08-2018, 12:37 PM

Checked, works fine. Conveniently. Thanks.

npr · 07-08-2018, 01:52 PM

Quote:

Originally Posted by geek1011

Thanks for the feedback! For making patch selection easier, what do you think of a web interface to generate kobopatch.yaml easier?

I think with the soft fail for "no" patches, all that's needed is to publish a standard kobopatch.yaml file listing all the patches flagged as "no". It's then a simple editing job for the user to change the "no" to "yes" for the patches he wants.

Can I also suggest adding a option to reduce the detail in the log file. Most of what's there means nothing to me, all I need is a list of patches successful processed.

For info: my kobopatch.yaml file

Spoiler:

Code:

## Works with kobopatch v0.6 or higher.
## You can update kobopatch by downloading the latest release from https://github.com/geek1011/kobopatch/releases. 
version: 4.9.11311
in: src/kobo-update-4.9.11311.zip
out: KoboRoot.tgz
log: log.txt

## The patch format to use: kobopatch (.yaml) or patch32lsb(.patch)
patchFormat: patch32lsb

## This section lists the patch files and the corresponding binary in the tgz.
patches:
  src/nickel.patch: usr/local/Kobo/nickel
  src/nickel.custom.patch: usr/local/Kobo/nickel
  src/libadobe.so.patch: usr/local/Kobo/libadobe.so
  src/libnickel.so.1.0.0.patch: usr/local/Kobo/libnickel.so.1.0.0
  src/librmsdk.so.1.0.0.patch: usr/local/Kobo/librmsdk.so.1.0.0

## You can put lines in the following section to override the enabled state of patches.
## The indentation matters! Each override should be indented by 4 spaces. Add to the 
## section below. This section can be copy and pasted into newer patch versions to
## keep your selections.
##
## Example of how it should look:
## overrides:
##   src/nickel.patch:
##     Custom synopsis/details line spacing: yes
##     Whatever the patch is called: no
##   src/libadobe.so.patch:
##     You get the idea: yes
overrides:
  src/nickel.patch:
     Disable reading footer: no
     Custom reading footer style: yes
     Custom synopsis/details line spacing: no
     Custom synopsis/font size: no
     Custom Header menubar - reduce height by 33%: yes
     Custom Header menubar - reduce height by 50%: no
     Dictionary pop-up frame size increase: no
     New home screen increasing cover size: no
     Reading stats/Author name cut when the series is showing bug fix: no
##     Increase size of Kepub chapter progress chart: no
##     New home screen removing the footer (row3) and increasing cover size: yes
##     show games on Beta Features: yes
  src/nickel.custom.patch:
     show games on Beta Features: yes 
  src/libadobe.so.patch:
     Remove PDF map widget shown during panning: no
  src/libnickel.so.1.0.0.patch:
     My 10 line spacing values: yes
     My 24 line spacing values: no
     Custom left & right margins: yes
     Custom font sizes: yes
     Freedom to advanced fonts control: yes
     ePub fixed/adjustable top/bottom margins: no
     ePub fixed top/bottom margins: no
     ePub disable built-in body padding-bottom: no
     Custom kepub default margins: yes
     Block WiFi firmware upgrade: no
     Custom footer (page number text): no
     Custom Sleep/Power-off timeouts: no
     Set KePub hyphenation: yes
     Fix three KePub fullScreenReading bugs: yes
     Force user line spacing in KePubs: yes
     Force user line spacing in ePubs (part 1 of 2): no
     Un-force font-family override p tags (std epubs): no
     Force user font-family in ePubs (Part 1 of 2): no
     ePub constant font sharpness: no
     KePub constant font sharpness: no
     Un-Force user text-align in div,p tags in KePubs: no
     Always display chapter name on navigation menu: no
     Un-Force user font-family in KePubs: no
     Un-force link decoration in KePubs: no
     KePub stylesheet additions: yes
     Change dicthtml strings to micthtml: no
     Allow searches on Extra dictionaries: no
     Ignore .otf fonts: no
     Brightness fine control: no
     Dictionary text font-family/font-size/line-height - beta: no
     Clock display duration: no
##     Keyboard template (Mini/Touch/Glo/Aura): no
##     Keyboard template (AuraHD/H2O): no
##     Keyboard template (GloHD/AuraOne/H2O2): no
##     Cyrillic keyboard (Mini/Touch/Glo/Aura/AuraHD/H2O): no
##     Cyrillic Keyboard (GloHD/AuraOne/H2O2): no
##     Arabic keyboard (Mini/Touch/Glo/Aura/AuraHD/H2O): no
##     Hebrew keyboard (Mini/Touch/Glo/Aura): no
##     Hebrew keyboard (AuraHD/H2O): no
##     Hebrew keyboard (GloHD/AuraOne/H2O2): no
##     Greek Keyboard (GloHD/AuraOne/H2O2): no
  src/librmsdk.so.1.0.0.patch:
     Disable orphans/widows avoidance: yes
     Default ePub monospace font (Courier): no
     Default ePub serif font (Amasis): no
     Default ePub sans-serif font (Gill Sans): no
     Default ePub symbol font (Symbol): no
     Force user line spacing in ePubs (Part 2 of 2): no
     Force user font-family in ePubs (Part 2 of 2): no
     Ignore ePub book Adobe XPGT stylesheet (page-template.xpgt): no
     Ignore ePub book CSS and Adobe XPGT stylesheets: no
     Ignore ePub TOC navpoints: no

Sorry, lost the indentation on pasting to the forum.
## The indentation matters! Each override should be indented by 4 spaces.
Corrected --- Thanks PeterT

PeterT · 07-08-2018, 02:15 PM

Quote:

Originally Posted by npr

I think with the soft fail for "no" patches, all that's needed is to publish a standard kobopatch.yaml file listing all the patches flagged as "no". It's then a simple editing job for the user to change the "no" to "yes" for the patches he wants.

Can I also suggest adding a option to reduce the detail in the log file. Most of what's there means nothing to me, all I need is a list of patches successful processed.

For info: my kobopatch.yaml file

Spoiler:

Code:

## Works with kobopatch v0.6 or higher.
## You can update kobopatch by downloading the latest release from https://github.com/geek1011/kobopatch/releases. 
version: 4.9.11311
in: src/kobo-update-4.9.11311.zip
out: KoboRoot.tgz
log: log.txt

## The patch format to use: kobopatch (.yaml) or patch32lsb(.patch)
patchFormat: patch32lsb

## This section lists the patch files and the corresponding binary in the tgz.
patches:
  src/nickel.patch: usr/local/Kobo/nickel
  src/nickel.custom.patch: usr/local/Kobo/nickel
  src/libadobe.so.patch: usr/local/Kobo/libadobe.so
  src/libnickel.so.1.0.0.patch: usr/local/Kobo/libnickel.so.1.0.0
  src/librmsdk.so.1.0.0.patch: usr/local/Kobo/librmsdk.so.1.0.0

## You can put lines in the following section to override the enabled state of patches.
## The indentation matters! Each override should be indented by 4 spaces. Add to the 
## section below. This section can be copy and pasted into newer patch versions to
## keep your selections.
##
## Example of how it should look:
## overrides:
##   src/nickel.patch:
##     Custom synopsis/details line spacing: yes
##     Whatever the patch is called: no
##   src/libadobe.so.patch:
##     You get the idea: yes
overrides:
  src/nickel.patch:
     Disable reading footer: no
     Custom reading footer style: yes
     Custom synopsis/details line spacing: no
     Custom synopsis/font size: no
     Custom Header menubar - reduce height by 33%: yes
     Custom Header menubar - reduce height by 50%: no
     Dictionary pop-up frame size increase: no
     New home screen increasing cover size: no
     Reading stats/Author name cut when the series is showing bug fix: no
##     Increase size of Kepub chapter progress chart: no
##     New home screen removing the footer (row3) and increasing cover size: yes
##     show games on Beta Features: yes
  src/nickel.custom.patch:
     show games on Beta Features: yes 
  src/libadobe.so.patch:
     Remove PDF map widget shown during panning: no
  src/libnickel.so.1.0.0.patch:
     My 10 line spacing values: yes
     My 24 line spacing values: no
     Custom left & right margins: yes
     Custom font sizes: yes
     Freedom to advanced fonts control: yes
     ePub fixed/adjustable top/bottom margins: no
     ePub fixed top/bottom margins: no
     ePub disable built-in body padding-bottom: no
     Custom kepub default margins: yes
     Block WiFi firmware upgrade: no
     Custom footer (page number text): no
     Custom Sleep/Power-off timeouts: no
     Set KePub hyphenation: yes
     Fix three KePub fullScreenReading bugs: yes
     Force user line spacing in KePubs: yes
     Force user line spacing in ePubs (part 1 of 2): no
     Un-force font-family override p tags (std epubs): no
     Force user font-family in ePubs (Part 1 of 2): no
     ePub constant font sharpness: no
     KePub constant font sharpness: no
     Un-Force user text-align in div,p tags in KePubs: no
     Always display chapter name on navigation menu: no
     Un-Force user font-family in KePubs: no
     Un-force link decoration in KePubs: no
     KePub stylesheet additions: yes
     Change dicthtml strings to micthtml: no
     Allow searches on Extra dictionaries: no
     Ignore .otf fonts: no
     Brightness fine control: no
     Dictionary text font-family/font-size/line-height - beta: no
     Clock display duration: no
##     Keyboard template (Mini/Touch/Glo/Aura): no
##     Keyboard template (AuraHD/H2O): no
##     Keyboard template (GloHD/AuraOne/H2O2): no
##     Cyrillic keyboard (Mini/Touch/Glo/Aura/AuraHD/H2O): no
##     Cyrillic Keyboard (GloHD/AuraOne/H2O2): no
##     Arabic keyboard (Mini/Touch/Glo/Aura/AuraHD/H2O): no
##     Hebrew keyboard (Mini/Touch/Glo/Aura): no
##     Hebrew keyboard (AuraHD/H2O): no
##     Hebrew keyboard (GloHD/AuraOne/H2O2): no
##     Greek Keyboard (GloHD/AuraOne/H2O2): no
  src/librmsdk.so.1.0.0.patch:
     Disable orphans/widows avoidance: yes
     Default ePub monospace font (Courier): no
     Default ePub serif font (Amasis): no
     Default ePub sans-serif font (Gill Sans): no
     Default ePub symbol font (Symbol): no
     Force user line spacing in ePubs (Part 2 of 2): no
     Force user font-family in ePubs (Part 2 of 2): no
     Ignore ePub book Adobe XPGT stylesheet (page-template.xpgt): no
     Ignore ePub book CSS and Adobe XPGT stylesheets: no
     Ignore ePub TOC navpoints: no

Sorry, lost the indentation on pasting to the forum.
## The indentation matters! Each override should be indented by 4 spaces.

Just add [code] .... [/code] to preserve indentation

geek1011 · 07-09-2018, 11:09 AM

Zlib support is coming very soon (https://github.com/geek1011/kobopatch/issues/4). I just figured out the biggest issue I was having.

The instructions I will be implementing are:

FindZlib: Finds a zlib css offset, not sensitive to line breaks or whitespace differences
- FindZlib: "#StatsView[qApp_deviceIsTrilogy=true]"

FindZlibHash: Finds a zlib css string by it's SHA1 hash
- FindZlib: w4ujrwe4ujtowe485rwoe4j (I know that isn't a valid hash)

ReplaceZlib: Replaces a zlib css string at the current offset (found with above functions or set with BaseAddress)
- ReplaceZlib:
Offset: 0
Find: "margin: 20px"
Replace: "margin: 40px"

In the future, I might also implement the following instruction:
SetZlibCSS: Sets a property for a CSS selector
- SetZlibCSS:
Offset: 0
Select: "BookProgressBarWidget[qApp_deviceIsTrilogy=true]"
Set: "min-height: 32px"

Please tell me what you think of those instructions or if you have any better/more useful ideas for instructions. Also, to save me some time for testing, it would be nice if oren64 could convert a simple patch to one using ReplaceZlib.

jackie_w · 07-09-2018, 01:31 PM

@geek1011,

I have a basic question about FindZlib. Using your above example:

If "#StatsView[qApp_deviceIsTrilogy=true]" is the correct unique string for this CSS stream but it is not the first line of the stream can you still use this function or do you need to use FindZlibHash instead?

geek1011 · 07-09-2018, 01:42 PM

Quote:

Originally Posted by jackie_w

@geek1011,

I have a basic question about FindZlib. Using your above example:

If "#StatsView[qApp_deviceIsTrilogy=true]" is the correct unique string for this CSS stream but it is not the first line of the stream can you still use this function or do you need to use FindZlibHash instead?

You can still use it even if it is not the first line. I recommend only using the hash if there is no unique string or if it is too long.

I'm almost finished now, I just need to test it and get it cross-compiling for macOS.

geek1011 · 07-09-2018, 04:07 PM

I have released v0.7.0 with zlib support.
I have implemented the instructions detailed above. For the old format, the instructions are called find_zlib, find_zlib_hash, and replace_zlib.

Please port your old-style zlib patches over and test them. You can post converted patches here. Once they are stable, please post them in this thread.

Also, you can use my new tool, cssextract, to extract the css from nickel with the information needed (offsets, hashes, css, etc) to make the patches for kobopatch.

If you have any questions, feel free to ask. Also, if you have any suggestions for new zlib instructions, I'd love to know.

07-08-2018, 12:37 PM	#69
Semwize Guru Posts: 891 Karma: 270670 Join Date: Jun 2016 Device: Kobo	Checked, works fine. Conveniently. Thanks. Last edited by Semwize; 07-10-2018 at 11:17 AM.

07-09-2018, 11:09 AM	#72
geek1011 Wizard Posts: 2,792 Karma: 6990707 Join Date: May 2016 Location: Ontario, Canada Device: Kobo Mini, Aura Edition 2 v1, Clara HD	Zlib support is coming very soon (https://github.com/geek1011/kobopatch/issues/4). I just figured out the biggest issue I was having. The instructions I will be implementing are: FindZlib: Finds a zlib css offset, not sensitive to line breaks or whitespace differences - FindZlib: "#StatsView[qApp_deviceIsTrilogy=true]" FindZlibHash: Finds a zlib css string by it's SHA1 hash - FindZlib: w4ujrwe4ujtowe485rwoe4j (I know that isn't a valid hash) ReplaceZlib: Replaces a zlib css string at the current offset (found with above functions or set with BaseAddress) - ReplaceZlib: Offset: 0 Find: "margin: 20px" Replace: "margin: 40px" In the future, I might also implement the following instruction: SetZlibCSS: Sets a property for a CSS selector - SetZlibCSS: Offset: 0 Select: "BookProgressBarWidget[qApp_deviceIsTrilogy=true]" Set: "min-height: 32px" Please tell me what you think of those instructions or if you have any better/more useful ideas for instructions. Also, to save me some time for testing, it would be nice if oren64 could convert a simple patch to one using ReplaceZlib. Last edited by geek1011; 07-09-2018 at 11:13 AM.

07-09-2018, 04:07 PM	#75
geek1011 Wizard Posts: 2,792 Karma: 6990707 Join Date: May 2016 Location: Ontario, Canada Device: Kobo Mini, Aura Edition 2 v1, Clara HD	I have released v0.7.0 with zlib support. I have implemented the instructions detailed above. For the old format, the instructions are called find_zlib, find_zlib_hash, and replace_zlib. Please port your old-style zlib patches over and test them. You can post converted patches here. Once they are stable, please post them in this thread. Also, you can use my new tool, cssextract, to extract the css from nickel with the information needed (offsets, hashes, css, etc) to make the patches for kobopatch. If you have any questions, feel free to ask. Also, if you have any suggestions for new zlib instructions, I'd love to know. Last edited by geek1011; 07-09-2018 at 04:56 PM.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
kobopatch - A new (WIP) patching system	geek1011	Kobo Developer's Corner	93	05-27-2018 04:04 PM
Patching H2O	Maillemom	Kobo Developer's Corner	8	10-19-2016 10:58 AM
Instructions for patching firmware 3.17.3	GeoffR	Kobo Developer's Corner	60	09-17-2015 06:35 PM
Instructions for patching firmware 3.8.0	GeoffR	Kobo Developer's Corner	92	10-22-2014 08:22 AM
how to clean more disk space in root file system to upgrade system	chinaet	iRex	1	12-18-2006 04:54 PM

07-07-2018, 05:11 PM	#63
npr Zealot Posts: 105 Karma: 37418 Join Date: Jan 2013 Device: Kobo Glo, Aura, H2OE2, Libra 2	Updated my H2O e2 to the latest firmware using this patching system. Everything went well, took a while to understand how to use the kobopatch.yaml file but expect that to save lots of time on future updates. Thanks for this new system it looks a excellent step forward. One additional thing I would like to be able to do is replace the hyphen.en_GB file with my own custom one during the patching process. Is this possible?

07-08-2018, 08:27 AM	#65
npr Zealot Posts: 105 Karma: 37418 Join Date: Jan 2013 Device: Kobo Glo, Aura, H2OE2, Libra 2	It's really not that difficult to understand, my main problem was entering the list of patches with the yes / no flag and the inevitable typos I made which stopped the patching process and needed correcting. The second issue I had was when I ran the new patch using the kobopatch.yaml from the previous patch version. The patching process stopped at every missing patch even though they were flagged as "no", it then took some time to identify the missing patch and remove it from the .yaml file. One solution would be to only list "patch name: yes" in the .yalm file Alternatively what do you think about making missing "no" patches have a soft fail. I don't see why the patch process needs to stop when a "no" patch is missing.

07-09-2018, 01:31 PM	#73
jackie_w Grand Sorcerer Posts: 6,224 Karma: 16536676 Join Date: Sep 2009 Location: UK Device: Kobo: KA1, ClaraHD, Forma, Libra2, Clara2E. PocketBook: TouchHD3	@geek1011, I have a basic question about FindZlib. Using your above example: If "#StatsView[qApp_deviceIsTrilogy=true]" is the correct unique string for this CSS stream but it is not the first line of the stream can you still use this function or do you need to use FindZlibHash instead?

Advert

Advert