We should probably just add unzip explicitly to
https://github.com/dnanexus/nucleus/blob/master/execserver/worker/lxc_default_packages.json,
since dealing with its absence is a big pain for no good reason.

On Mon, Apr 15, 2013 at 2:20 PM, Phil Sung <notifications@github.com> wrote:

> In resources/usr/local/lib/dx_fastq_to_reads.py:
>
> > @@ -84,6 +86,8 @@ def unpack_and_open(input):
> >      uncomp_util = None
> >      if file_type == 'XZ compressed data':
> >          uncomp_util = 'xzcat'
> > +    elif file_type.startswith("Zip archive data"):
> > +        uncomp_util = 'unzip -p'
>
> You're right, it is in fact ok to use unzip this way for the time being.
> It looks like we recently started installing r-base which transitively
> pulls in unzip.
>
> If we ever made R support some kind of optional add-on (not totally
> inconceivable), unzip would stop getting installed by default, so I think
> it would be a good idea to explicitly add it to the execDepends.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dnanexus/reads_importer/pull/1/files#r3802143>
> .
>