Date: Sat, 31 Jul 2010 13:33:04 -0700 From: Garrett Cooper <gcooper@FreeBSD.org> To: Julien Laffaye <jlaffaye@freebsd.org> Cc: Perforce Change Reviews <perforce@freebsd.org> Subject: Re: PERFORCE change 181582 for review Message-ID: <AANLkTimOB1u9eF9JrFyARY6R2owgDUKwESeHqrC-b6yK@mail.gmail.com> In-Reply-To: <201007292123.o6TLN3Td039516@repoman.freebsd.org> References: <201007292123.o6TLN3Td039516@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jul 29, 2010 at 2:23 PM, Julien Laffaye <jlaffaye@freebsd.org> wrot= e: > http://p4web.freebsd.org/@@181582?ac=3D10 > > Change 181582 by jlaffaye@jlaffaye-chulak on 2010/07/29 21:23:01 > > =A0 =A0 =A0 =A0- Rework the fetch_archive() function to return a file des= criptor of the > =A0 =A0 =A0 =A0downloaded archive. > =A0 =A0 =A0 =A0Reading on-the-fly doesn't work if we fetch dependencies: = the remote > =A0 =A0 =A0 =A0server close the connection because we are inactive (in fa= ct we are > =A0 =A0 =A0 =A0busy downloading the dependency). Reading on-the-fly could= work if we > =A0 =A0 =A0 =A0knew the dependencies, in the proper order, of the package= before > =A0 =A0 =A0 =A0downloading it (thus, download and install the dependencie= s first). > =A0 =A0 =A0 =A0- While I'm here, print the progress (percentage) of the d= owload. > ... > + =A0 =A0 =A0 if (archive_read_open_fd(a, fd, 10240) !=3D ARCHIVE_OK) { > + =A0 =A0 =A0 =A0 =A0 warnx("archive_read_open_fd(): %s", archive_error_s= tring(a)); > + =A0 =A0 =A0 =A0 =A0 retcode =3D 1; > + =A0 =A0 =A0 =A0 =A0 goto cleanup; Overall change looks good, but why 10240 :)? -Garrett
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTimOB1u9eF9JrFyARY6R2owgDUKwESeHqrC-b6yK>