search

fatty- / daisy-pipeline

Automatically exported from code.google.com/p/daisy-pipeline
0 stars 0 forks source link

dtbook-to-pef: external resources are required #329

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Run the sample DTBook through dtbook-to-pef, but only post hauy_valid.xml
2. The script will fail because the image is missing

What is the expected output? What do you see instead?
For dtbook-to-pef, images are not used in the output but are required in the 
input.

What if we start issuing warnings instead of errors for missing resources in 
conversion scripts unless we actually access the resource files (load them into 
memory or store/copy them)?

Validation (with errors) of whether all referenced resources are present can be 
done when storing the filesets (px:zedai-store, px:pef-store, etc).

Original issue reported on code.google.com by josteinaj@gmail.com on 19 Jun 2013 at 1:46

GoogleCodeExporter commented 9 years ago

Original comment by josteinaj@gmail.com on 19 Jun 2013 at 1:52

GoogleCodeExporter commented 9 years ago 
Maybe we could have an option in DTBook to ZedAI to ignore whether external 
resources like images are present on disk. What do you think?

Original comment by marisa.d...@gmail.com on 19 Jun 2013 at 6:40

GoogleCodeExporter commented 9 years ago 
Would you want to expose it to the end-user (px:dtbook-to-zedai) or just
for the script author (px:zedai-store)? Is there a use-case for end-users
to disable checking for external resources in dtbook-to-zedai?
Den 19. juni 2013 20:40 skrev <daisy-pipeline@googlecode.com> følgende:

Original comment by josteinaj@gmail.com on 19 Jun 2013 at 6:58

GoogleCodeExporter commented 9 years ago 
I think we should expose it to the end-user (though it would be off by 
default). It probably suits someone's workflow, somewhere.

Original comment by marisa.d...@gmail.com on 19 Jun 2013 at 7:04

GoogleCodeExporter commented 9 years ago 
"off" as in "don't ignore" (fail on not found) by default, right?
Den 19. juni 2013 21:04 skrev <daisy-pipeline@googlecode.com> følgende:

Original comment by josteinaj@gmail.com on 19 Jun 2013 at 7:08

GoogleCodeExporter commented 9 years ago 
yep exactly

Original comment by marisa.d...@gmail.com on 19 Jun 2013 at 7:10

GoogleCodeExporter commented 9 years ago 
Sounds good
Den 19. juni 2013 21:10 skrev <daisy-pipeline@googlecode.com> følgende:

Original comment by josteinaj@gmail.com on 19 Jun 2013 at 7:14

GoogleCodeExporter commented 9 years ago 
Fixed:
https://github.com/daisy-consortium/pipeline-scripts/commit/0ad20f30fdcb313916ec
d8ca7eb41a4145859cd8

Original comment by marisa.d...@gmail.com on 7 Jul 2013 at 1:32