Skip to content
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

Failed to merge #74

Closed
LianAFei opened this issue Apr 3, 2019 · 3 comments
Closed

Failed to merge #74

LianAFei opened this issue Apr 3, 2019 · 3 comments

Comments

@LianAFei
Copy link

LianAFei commented Apr 3, 2019

hi @hhrutter
When I use the function MergeCommand in this great great project,something wrong with the result.
Use the test files, I could get the merged file,but There is no merged file after used the function with my pdf files.
And It shows me "DereferenceName: wrong type " with the command pdfcpu.exe merge.
There are the files which failed to merge.
test4.pdf
test5.pdf
test6.pdf

Hope your replay.

@hhrutter
Copy link
Collaborator

hhrutter commented Apr 3, 2019

Thank you for reporting this. I will check it out.

@hhrutter
Copy link
Collaborator

hhrutter commented Apr 3, 2019

The InfoDict has an attribute called Trapped. This should be a Name object.
In the files you provided Trapped is a Boolean. Here is the part of the spec that explicitly forbids Boolean:

Screenshot 2019-04-04 at 00 13 04

Since all your files open in Acrobat Reader I extended relaxed validation for this issue.

Long story short - You are fine if you build from the latest commit.
;)

Go-> ~/Documents/pdf/out: pdfcpu merge out.pdf test?.pdf
merging into out.pdf: [test4.pdf test5.pdf test6.pdf]
merging in test5.pdf ...
merging in test6.pdf ...
writing out.pdf ...

Have fun with pdfcpu!
💚

@LianAFei
Copy link
Author

LianAFei commented Apr 4, 2019

Thanks for your beautiful work.@hhrutter
;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants