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
Add experimental PDF/A compliance mode #3269
base: master
Are you sure you want to change the base?
Conversation
lib/res/sRGB2014.icc
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you say where you got this file?
What licence it is?
Is this the same file as https://sources.debian.org/src/tcpdf/6.7.4%2Bdfsg-1/include/sRGB.icc/?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Let's let @bsweeney let us know if adding a licence file should be done like in https://github.com/dompdf/php-font-lib/tree/master/tests/resources/fonts/ahem
Co-authored-by: William Desportes <williamdes@wdes.fr>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The diff looks good, maybe some concerns about the licencing of the binary file
@Starfox64 do you think it would solve #3269 ? |
That's the wrong Issue ID I think. |
Indeed, this one: #3442 Could you rebase your work ? 🇫🇷 👋🏻 |
I've done a merge recently and it looks like I'm ahead of master. With regards to #3442 I think this might be fixed by this line https://github.com/dompdf/dompdf/pull/3269/files#diff-f365bcb24d5c081b6bfde4aa59a8eda68824ea9658c362098554fc659dcec1d0R3426 It's funny to see that we are literally trying to do the same thing btw, nothing spells out fun more than trying to embed XML in PDF metadata. |
Okayy, I got it wrong: |
https://github.com/williamdes/dompdf/tree/pdfa-2.0 and https://github.com/williamdes/dompdf/tree/feat/pdfa is a rebased version without merges |
FYI I'm re-targeting this to 3.1 (from 3.0.1) since it's a new feature not a patch. I know there are a lot of issues slate for 3.0.x releases but I suspect most of those will be pushed out once I have a chance to review them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add Fixes: #3442
to this PR ?
It passes nearly all compliance tests except
[Specification: ISO 19005-3:2012, Clause: 6.2.11.4.1, Test number: 1](https://github.com/veraPDF/veraPDF-validation-profiles/wiki/PDFA-Parts-2-and-3-rules#rule-621141-1)
The font programs for all fonts used for rendering within a conforming file shall be embedded within that file, as defined in ISO 32000-1:2008, 9.9. Failed
Let's follow up on #3443
This PR adds experimental PDF/A compliance mode and closes #1106
I'm calling this experimental for multiple reasons:
This still should work for most people on non esoteric documents.
Addendum
Fixes #3442