-
Notifications
You must be signed in to change notification settings - Fork 229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
There is no item named 'META-INF\\\\container.xml #162
Comments
Hi! Can you maybe provide the EPUB file so I could test it? Are you maybe using Windows? |
Yes, I'm using windows. |
Yeah, the problem is that ebooklib uses certain function calls like os.path.normpath to normalise the path but on the Windows machine it also changes directory separator used when working with ZipFile. ZipFile expects it to be "/" but normpath changes it to "\". I don't have a Windows machine (I just installed it in the VirtualBox but I don't have Visual Studio to compile the lxml) of my own but will change it on a branch and let you test it :) |
Sorry this took way too long but this should be fixed now and the changes are in master branch. You can test it now. I tried it only on Windows with samples 01 (to create EPUB) and 06 (to parse existing EPUB). |
Sorry it took me a while to test and reply. It works fine so far. Thanks for the quick response. |
Great! Will close this one for now. |
I have the same error, everything works OK on OSX, but not in Window7 32bit (Have not tried windows 64bit yet). How can I get that new version? I run ebooklib version 0.17. Thanks |
First time I changed a site-package myself in Python. Went into the python 3 environmental path in my Windows 7 PC, found site-packages directory, copy/past your updated util.py and epub.py , to replace the previous version. Bang and it works. |
Install latest version from GitHub until we do the release:
|
Hi Aerkalov,
Many thanks for this.
I guess I also can use that for OSX.
Because I see that those directory + containers file are missing, but it did not give a error and did work OK.
Best regards,
Jan
… On 3 Sep 2018, at 12:33, Aleksandar Erkalović ***@***.***> wrote:
Install latest version from GitHub until we do the release:
pip install -e git+git://github.com/aerkalov/ebooklib.git#egg=EbookLib
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Great! I have the same problem and |
Just a kind reminder that the git version is still not in the public repostory referenced by pip. |
@padmalcom What do you mean? Can you explain more. |
@aerkalov Hi Aleksandar, I mean that I still have to install ebooklib from git. When I install via pip and pypi I keep getting the error described above on windows. |
@padmalcom Must have been some error when I was uploading package. I just released never version 0.17.1 which should fix your problem. |
Hello, I ran into the same issue. It seems that if you install |
@Quemoy I have no experience with conda-forge but looking at it this would be the feedstock for the ebooklib and those Contributors would be the one who would know what to do - https://github.com/conda-forge/ebooklib-feedstock Looking at the project I would assume it is required to update the version number and change the sha256 - https://github.com/conda-forge/ebooklib-feedstock/blob/main/recipe/meta.yaml Not really sure how this all works but looking at the docs if you feel adventurous you might be able to update it also - https://conda-forge.org/#update_recipe |
I am receiving this error on v.018 with the following ebook Error: |
I am also seeing this issue. |
Python 3.5.1 32bit.
Cannot read epub files with epub.read_epub. Even if the epub is created using the example in documentations, I still get: "There is no item named 'META-INF\\container.xml" in the archive"
The text was updated successfully, but these errors were encountered: