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
Reviewer Review Type Date Requested Status
Jonathan Lange Approve
Review via email: mp+86757@code.launchpad.net

Commit message

Add extra_files_from_paths to all extra binary 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.
94. By James Westby

Target filename must be absolute path.

Revision history for this message
Jonathan Lange (jml) wrote :

Looks good to me. Would be good to have a docstring for FileFromDisk.

review: Approve
Revision history for this message
James Westby (james-w) wrote :

Added.

Thanks,

James

95. By James Westby

Add a docstring to FileFromDisk. Thanks jml.

Preview Diff

Empty

Subscribers

People subscribed via source and target branches

to all changes: