Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

also extract PDFs #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mayoff
Copy link

@mayoff mayoff commented Jul 30, 2016

No description provided.

@steventroughtonsmith
Copy link
Owner

But PDFs in Asset Catalogs are converted to bitmaps at compile-time, no? Is there a case in the OS where this is not true? Can you provide an example?

@mayoff
Copy link
Author

mayoff commented Jul 31, 2016

/Applications/Xcode.app/Contents/SharedFrameworks/DVTKit.framework/Resources/Assets.car (Xcode 7.3.1) has a bunch, which is why I wrote this patch.

@mayoff
Copy link
Author

mayoff commented Jul 31, 2016

More generally, Xcode includes the PDF in Assets.car when building a Mac app, in addition to the generated PNGs.

@steventroughtonsmith
Copy link
Owner

Perfect, thanks! I'll need to do some work before I merge this as the current seeds seem to break things somewhat. No immediate ETA

@mayoff
Copy link
Author

mayoff commented Jul 31, 2016

Yeah, I noticed it doesn't work on car files in Xcode 8b3.

@steventroughtonsmith
Copy link
Owner

Seems to be a locking issue, but not sure if I should fix it right now if it's just going to be a temporary bug in a few seeds or whatever. Will revisit closer to GM 👍

@getaaron
Copy link

@mayoff @steventroughtonsmith Wish I had seen this before I wrote #24.

Some elements from this PR are definitely more elegant than mine. I think you may need the [CUICatalog -initWithURL: error:] initializer I used in my PR to get PDFs out of the Assets.car file. It was crashing for me using the older CUIThemeFacet approach.

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

Successfully merging this pull request may close these issues.

3 participants