![]() |
#1456 | |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,771
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
Quote:
If it still doesn't work, download the portable 5.0.1 and try that in case it could be something in your settings. Did you have any old PY2 plugins installed when you upgraded to 5.x? |
|
![]() |
![]() |
![]() |
#1457 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,447
Karma: 8012886
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
|
This looks like another example of a binary file inside the epub, such as an image, that the plugin isn't prepared for. I won't be able to debug it without the book. Please send a copy to calibre-bugs-mod [at] charleshaley [dot] org (make the replacements). I promise I won't keep it or otherwise violate copyright.
|
![]() |
![]() |
![]() |
#1458 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,447
Karma: 8012886
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
|
Yet another attempt ...
I found and fixed a place where unneeded binary files were being read, causing the plugin to fail. After fixing this the plugin runs on both calibre 4 and calibre 5 (for me), tested using several of my books & JSWolf's test book. All boxes were checked. We might be able to avoid a fork. ![]() Last edited by chaley; 09-28-2020 at 05:22 PM. Reason: Remove obsolete install file |
![]() |
![]() |
![]() |
#1459 |
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,055
Karma: 11391181
Join Date: Mar 2013
Location: Guben, Brandenburg, Germany
Device: Kobo Clara 2E, Tolino Shine 3
|
Thank you, chaley, it works on my Windows 10, 64bit. I'll try it on the 32bit as well.
I suppose your request from #1457 is obsolete? |
![]() |
![]() |
![]() |
#1460 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,447
Karma: 8012886
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
|
|
![]() |
![]() |
![]() |
#1461 |
Wizard
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 1,055
Karma: 11391181
Join Date: Mar 2013
Location: Guben, Brandenburg, Germany
Device: Kobo Clara 2E, Tolino Shine 3
|
Works on my 32bit as well! Great!
|
![]() |
![]() |
![]() |
#1462 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,771
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
I've been testing the new beta (1.5.6) and it's working with Calibre 5. I'll test with 4.23 soon.
Last edited by JSWolf; 09-28-2020 at 06:13 AM. |
![]() |
![]() |
![]() |
#1463 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 976
Karma: 3475832
Join Date: May 2012
Location: France
Device: Elipsa, Sage, Libra 2
|
It works for me, calibre 5, windows 10 version 2004, 64 bits.
|
![]() |
![]() |
![]() |
#1464 |
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 46,280
Karma: 169098402
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
@chaley: Ran it against 8 books with every option checked and then one option at a time and no issues when running epubcheck against the modified epubs. Looks good. I also ran it against a total crap epub and it didn't do any damage to it (one of my wife's books that was saved to epub from a word processor -- it makes a great test for epubcheck.
![]() Again, thanks very much for your time and effort! |
![]() |
![]() |
![]() |
#1465 |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,771
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
Looks like we are good to go.
Thank you very much chaley! ![]() |
![]() |
![]() |
![]() |
#1466 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,447
Karma: 8012886
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
|
|
![]() |
![]() |
![]() |
#1467 |
Connoisseur
![]() Posts: 52
Karma: 10
Join Date: Jun 2012
Location: Zaandam, The Netherlands
Device: sony PRS-T1 / Kobo Glo HD
|
I have tested a few books with version 1.5.6 (calibre 5.01) . As far as I can see the plugin works ok.
One small observation: the help file gives an error message on line 159: File "calibre_plugins.modify_epub.dialogs", line 159, in _help_link_activated File "calibre_plugins.modify_epub.action", line 138, in show_help File "calibre_plugins.modify_epub.action", line 136, in get_help_file_resource TypeError: write() argument must be str, not bytes Thanks for the conversion of the plugin. I use it a lot |
![]() |
![]() |
![]() |
#1468 | |
Bibliophagist
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 46,280
Karma: 169098402
Join Date: Jul 2010
Location: Vancouver
Device: Kobo Sage, Libra Colour, Lenovo M8 FHD, Paperwhite 4, Tolino epos
|
Quote:
![]() Spoiler:
|
|
![]() |
![]() |
![]() |
#1469 |
Grand Sorcerer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 12,447
Karma: 8012886
Join Date: Jan 2010
Location: Notts, England
Device: Kobo Libra 2
|
Thank you @fvdham and @DNSB.
Version 1.5.7, fixing the problem accessing the help file. Tested on 4.23 and 5.0.1. If
Last edited by chaley; 09-29-2020 at 06:11 AM. Reason: Remove obsolete plugin zip |
![]() |
![]() |
![]() |
#1470 | |
Resident Curmudgeon
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 79,771
Karma: 145864619
Join Date: Nov 2006
Location: Roslindale, Massachusetts
Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3
|
Quote:
With 4.23 (all options checked)and the ePub I sent to you that's not scrambled. Code:
calibre, version 4.23.0 ERROR: Modify ePub failed: No ePub files were updated Modify ePubs Logfile for book ID 2252 (The Measure of the Magic / Terry Brooks) 2252 Modifying: C:\Users\help\AppData\Local\Temp\calibre_eomo94\7ggd8n_modify_epub\2252.epub Updating metadata and cover Parsing xml file: Broo_9780345529213_epub_opf_r1.opf Parsing xml file: Broo_9780345529213_epub_ncx_r1.ncx Looking for redundant entries in manifest Looking for unmanifested files Found file to to add: META-INF/com.apple.ibooks.display-options.xml The Measure of the Magic - ERROR: Traceback (most recent call last): File "calibre_plugins.modify_epub.modify", line 79, in process_book File "calibre_plugins.modify_epub.modify", line 129, in _process_book File "calibre_plugins.modify_epub.modify", line 282, in _process_unmanifested_files File "calibre_plugins.modify_epub.container", line 404, in add_name_to_manifest File "calibre_plugins.modify_epub.container", line 417, in generate_manifest_id OverflowError: Python int too large to convert to C long ePub not changed after 1.39 seconds Last edited by JSWolf; 09-28-2020 at 06:24 PM. |
|
![]() |
![]() |
![]() |
Tags |
modify epub |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
[GUI Plugin] Quality Check | kiwidude | Plugins | 1251 | 07-07-2025 09:13 PM |
[GUI Plugin] Open With | kiwidude | Plugins | 404 | 02-21-2025 05:42 AM |
[GUI Plugin] Manage Series | kiwidude | Plugins | 167 | 07-28-2024 03:07 PM |
Modify ePub plugin dev thread | kiwidude | Development | 346 | 09-02-2013 05:14 PM |
[GUI Plugin] Plugin Updater **Deprecated** | kiwidude | Plugins | 159 | 06-19-2011 12:27 PM |