Small bug report

Turtle91 · 07-19-2017, 01:02 PM

Found a bug-ish.

When you try and merge a file into the previous file by right clicking on the filename in the book browser and selecting merge - or typing ctl-m - I get an "Embedded Python Error". With a "Error: traceback report is missing" detail.

When you close/cancel the error report the merge is completed properly.

Windows 10 64, Sigil 0.9.7

KevinH · 07-19-2017, 01:29 PM

I just tried to recreate this with current builds from github master on a Mac and can not recreate what you are reporting at all. The merge works just fine with no "Embedded Python Error".

So I can not recreate this on a Mac at the very least.

Can anyone else confirm the bug? Is it a Windows only bug?

Thanks,

KevinH

Quote:

Originally Posted by Turtle91

Found a bug-ish.

When you try and merge a file into the previous file by right clicking on the filename in the book browser and selecting merge - or typing ctl-m - I get an "Embedded Python Error". With a "Error: traceback report is missing" detail.

When you close/cancel the error report the merge is completed properly.

Windows 10 64, Sigil 0.9.7

DiapDealer · 07-19-2017, 01:33 PM

Must be something specific to the xhtml in @Turtle91's epub. I merge file in exactly the manner he described all the time (on Windows and Linux) without an error. Did it just now.

But of course, I'm using the latest version of Sigil and not 0.9.7 (though I don't remember getting the errors on older versions of Sigil, either).

DiapDealer · 07-19-2017, 01:53 PM

Never mind, I know exactly what this is. It occurs whenever splits or merges are initiated when the opf tab is open. So long as only xhtml tabs are currently open, there is no error when splitting/merging.

I know because I explicitly fixed the issue in Sigil v0.9.8.

KevinH · 07-19-2017, 01:55 PM

Please Ignore the below as DiapDealer has already fixed the issue in Sigil 0.9.8 which is why I could not recreate it in master.

Turtle91,
On a copy of the problem file, would you please run Sigil->Tools->Reformat HTML-> "Mend All HTML Files" just **before** trying your problem merge and let me know if the issue goes away.

That way we can rule out non-well-formed xhtml from causing the problem.

Please let us know what you see.

Turtle91 · 07-19-2017, 07:29 PM

I feel so bad!!

I normally update to the latest as soon as it is released...didn't even think of that aspect. I'll update and check again.

JSWolf · 07-19-2017, 08:33 PM

Bug reports don't count unless they are from the latest version.

DiapDealer · 07-19-2017, 10:09 PM

No harm no foul.

BetterRed · 07-19-2017, 10:37 PM

Quote:

Originally Posted by Turtle91

I feel so bad!!

I normally update to the latest as soon as it is released...didn't even think of that aspect. I'll update and check again.

Don't feel bad, version 0.9.8 also snuck past me, I only realised I was a release behind last week. I've now put the releases page into my RSS sucker-upper.

BR

Turtle91 · 07-19-2017, 11:14 PM

Yup - that was it. All fixed!

Hitch · 07-21-2017, 07:15 PM

Oh, I love a good HEA ending. :-)

Hitch

07-19-2017, 01:02 PM	#1
Turtle91 A Hairy Wizard Posts: 3,295 Karma: 20171067 Join Date: Dec 2012 Location: Charleston, SC today Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire	Small bug report Found a bug-ish. When you try and merge a file into the previous file by right clicking on the filename in the book browser and selecting merge - or typing ctl-m - I get an "Embedded Python Error". With a "Error: traceback report is missing" detail. When you close/cancel the error report the merge is completed properly. Windows 10 64, Sigil 0.9.7

07-19-2017, 01:53 PM	#4
DiapDealer Grand Sorcerer Posts: 28,341 Karma: 203719646 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Never mind, I know exactly what this is. It occurs whenever splits or merges are initiated when the opf tab is open. So long as only xhtml tabs are currently open, there is no error when splitting/merging. I know because I explicitly fixed the issue in Sigil v0.9.8. Last edited by DiapDealer; 07-19-2017 at 02:03 PM.

Thread Tools	Search this Thread
Show Printable Version Email this Page	Search this Thread: Advanced Search

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
How to report a bug	osmin	Calibre	4	06-20-2017 05:56 PM
very small bug report	cybmole	Calibre Companion	10	05-22-2014 06:21 AM
Bug report	mtravellerh	PocketBook	303	11-03-2013 07:21 PM
How to report a bug???	tmlee51	Calibre	2	02-14-2012 11:07 PM

07-19-2017, 01:33 PM	#3
DiapDealer Grand Sorcerer Posts: 28,341 Karma: 203719646 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	Must be something specific to the xhtml in @Turtle91's epub. I merge file in exactly the manner he described all the time (on Windows and Linux) without an error. Did it just now. But of course, I'm using the latest version of Sigil and not 0.9.7 (though I don't remember getting the errors on older versions of Sigil, either).

07-19-2017, 01:55 PM	#5
KevinH Sigil Developer Posts: 8,439 Karma: 5703082 Join Date: Nov 2009 Device: many	Please Ignore the below as DiapDealer has already fixed the issue in Sigil 0.9.8 which is why I could not recreate it in master. Turtle91, On a copy of the problem file, would you please run Sigil->Tools->Reformat HTML-> "Mend All HTML Files" just before trying your problem merge and let me know if the issue goes away. That way we can rule out non-well-formed xhtml from causing the problem. Please let us know what you see.

07-19-2017, 07:29 PM	#6
Turtle91 A Hairy Wizard Posts: 3,295 Karma: 20171067 Join Date: Dec 2012 Location: Charleston, SC today Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire	I feel so bad!! I normally update to the latest as soon as it is released...didn't even think of that aspect. I'll update and check again.

07-19-2017, 08:33 PM	#7
JSWolf Resident Curmudgeon Posts: 78,958 Karma: 144284074 Join Date: Nov 2006 Location: Roslindale, Massachusetts Device: Kobo Libra 2, Kobo Aura H2O, PRS-650, PRS-T1, nook STR, PW3	Bug reports don't count unless they are from the latest version.

07-19-2017, 10:09 PM	#8
DiapDealer Grand Sorcerer Posts: 28,341 Karma: 203719646 Join Date: Jan 2010 Device: Nexus 7, Kindle Fire HD	No harm no foul.

07-19-2017, 11:14 PM	#10
Turtle91 A Hairy Wizard Posts: 3,295 Karma: 20171067 Join Date: Dec 2012 Location: Charleston, SC today Device: iPhone 15/11/X/6/iPad 1,2,Air & Air Pro/Surface Pro/Kindle PW & Fire	Yup - that was it. All fixed!

07-21-2017, 07:15 PM	#11
Hitch Bookmaker & Cat Slave Posts: 11,503 Karma: 158448243 Join Date: Apr 2010 Location: Phoenix, AZ Device: K2, iPad, KFire, PPW, Voyage, NookColor. 2 Droid, Oasis, Boox Note2	Oh, I love a good HEA ending. :-) Hitch

Advert

Advert