Merge lp:~james-w/pkgme/extra_files_from_paths into lp:pkgme
Proposed by
James Westby
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | James Westby | ||||
Approved revision: | 95 | ||||
Merged at revision: | 95 | ||||
Proposed branch: | lp:~james-w/pkgme/extra_files_from_paths | ||||
Merge into: | lp:pkgme | ||||
Diff against target: | 0 lines | ||||
To merge this branch: | bzr merge lp:~james-w/pkgme/extra_files_from_paths | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jonathan Lange | Approve | ||
Review via email: mp+86757@code.launchpad.net |
Commit message
Add extra_files_
Description of the change
Hi,
This allows us to provide icons in the binary backends, as extra_files
isn't usable with binary files.
I added to the XXX code that breaks the layering, but I couldn't come up
with a better strategy that wasn't also much more complex.
Another alternative would be to not read the paths in pkgme, but have
extra_files_base64 or something. This just seemed a bit easier for backends,
but it may be problematic. For instance the filename is assumed to be
absolute, and allowing relative paths would mean the InfoElement knowing
about the path that it is acting on.
Thanks,
James
To post a comment you must log in.
Looks good to me. Would be good to have a docstring for FileFromDisk.