<feed xmlns='http://www.w3.org/2005/Atom'>
<title>build_soong/zip, branch android-platform-11.0.0_r2</title>
<subtitle>Fork of build/soong
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/'/>
<entry>
<title>-l option soong_zip can accept a file having space separated list</title>
<updated>2019-11-04T05:23:07+00:00</updated>
<author>
<name>Jiyong Park</name>
<email>jiyong@google.com</email>
</author>
<published>2019-11-04T04:18:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=04bbf98e069c35282ab904c154356f3f9755da97'/>
<id>04bbf98e069c35282ab904c154356f3f9755da97</id>
<content type='text'>
-l option of soong_zip is used to specify the list of input files that
should be zipped. However, it only accepted newline-separated list in the
file. This prevented us from using a rspfile as paths in rspfile are
space-separated in a single line. Fixing the problem by splitting the
file content by any unicode whitespace character including newline and
space.

Test: zip_test
Change-Id: Iba572109e30c01540eacf20bd2794ba60c870fa3
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
-l option of soong_zip is used to specify the list of input files that
should be zipped. However, it only accepted newline-separated list in the
file. This prevented us from using a rspfile as paths in rspfile are
space-separated in a single line. Fixing the problem by splitting the
file content by any unicode whitespace character including newline and
space.

Test: zip_test
Change-Id: Iba572109e30c01540eacf20bd2794ba60c870fa3
</pre>
</div>
</content>
</entry>
<entry>
<title>Support moving sources in srcjars in soong_zip</title>
<updated>2019-06-18T20:33:20+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2019-06-17T21:12:41+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=9cb51dbc0b4fe0e60ec604f3fa1440014c669e13'/>
<id>9cb51dbc0b4fe0e60ec604f3fa1440014c669e13</id>
<content type='text'>
Add a -srcjar argument to soong_zip that causes it to read the
package statement of each .java file and use that to place the
source file at a path that matches the package.

Test: jar_test.go, zip_test.go
Change-Id: I36017e42445ba3b0a82a10a8d81e8ac0cca096f2
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a -srcjar argument to soong_zip that causes it to read the
package statement of each .java file and use that to place the
source file at a path that matches the package.

Test: jar_test.go, zip_test.go
Change-Id: I36017e42445ba3b0a82a10a8d81e8ac0cca096f2
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix soong_zip --help</title>
<updated>2019-04-21T04:45:51+00:00</updated>
<author>
<name>Dan Willemsen</name>
<email>dwillemsen@google.com</email>
</author>
<published>2019-04-21T04:44:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=7f276c6b772a1077dbe7c335830b1fdeed1d1919'/>
<id>7f276c6b772a1077dbe7c335830b1fdeed1d1919</id>
<content type='text'>
It was not printing out the flag descriptions because it was using the
default FlagSet instead of the custom FlagSet when calling
PrintDefaults.

Test: soong_zip --help
Change-Id: I8a5705f8701ce0c05dde144b9a14962faa384e65
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It was not printing out the flag descriptions because it was using the
default FlagSet instead of the custom FlagSet when calling
PrintDefaults.

Test: soong_zip --help
Change-Id: I8a5705f8701ce0c05dde144b9a14962faa384e65
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix soong_zip printing warnings with --ignore_missing_files</title>
<updated>2018-12-27T20:41:25+00:00</updated>
<author>
<name>Dan Willemsen</name>
<email>dwillemsen@google.com</email>
</author>
<published>2018-12-27T20:41:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=edc934ccc935e96ec83dc9932e03d4b23578a155'/>
<id>edc934ccc935e96ec83dc9932e03d4b23578a155</id>
<content type='text'>
We weren't defaulting args.Stderr to os.Stderr, so we had been calling
fmt.Fprintln with `nil`.

Test: soong_zip --ignore_missing_files -C test/ -f abc -o test.zip
Change-Id: I0421f4bfb07fa36edf86c68ffc847995777c8221
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We weren't defaulting args.Stderr to os.Stderr, so we had been calling
fmt.Fprintln with `nil`.

Test: soong_zip --ignore_missing_files -C test/ -f abc -o test.zip
Change-Id: I0421f4bfb07fa36edf86c68ffc847995777c8221
</pre>
</div>
</content>
</entry>
<entry>
<title>soong_zip: add --ignore_missing_files flag</title>
<updated>2018-09-28T23:14:52+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-28T22:16:48+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=4be8f9e2a3e857050f4d6d9b3d728492828445bf'/>
<id>4be8f9e2a3e857050f4d6d9b3d728492828445bf</id>
<content type='text'>
soong_zip builds a list of files to zip early and then starts
zipping them all.  If a directory being zipped is concurrently
modified, a file that existed when soong_zip started may not
still exist.  Add a flag that continues when an expected file
does not exist.  Print a warning, since this should be rare
in normal usages but is a sign of a problem if it happens
regularly.

Test: zip_test.go
Test: m checkbuild
Test: m platform
Change-Id: I78426fe66fded8528ddd436c0f71a7442183cfeb
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
soong_zip builds a list of files to zip early and then starts
zipping them all.  If a directory being zipped is concurrently
modified, a file that existed when soong_zip started may not
still exist.  Add a flag that continues when an expected file
does not exist.  Print a warning, since this should be rare
in normal usages but is a sign of a problem if it happens
regularly.

Test: zip_test.go
Test: m checkbuild
Test: m platform
Change-Id: I78426fe66fded8528ddd436c0f71a7442183cfeb
</pre>
</div>
</content>
</entry>
<entry>
<title>Add a --symlinks argument to soong_zip</title>
<updated>2018-09-28T21:01:33+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-21T22:12:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=09f11056f876a9f705a53fd6f122aa7575e0b51b'/>
<id>09f11056f876a9f705a53fd6f122aa7575e0b51b</id>
<content type='text'>
Add a --symlinks argument that defaults to true to soong_zip.
Passing --symlinks=false will cause it to follow symlinks instead
of storing them in the zip file.

Relands I4deb98daa9d4ba9f94e3d7670c117fe00381d2ba with tests.

Bug: 112843624
Test: glob_test.go
Test: zip_test.go
Test: m checkbuild
Change-Id: I0eff9c1f2dba79e873fda381ff585df55d5aaaad
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a --symlinks argument that defaults to true to soong_zip.
Passing --symlinks=false will cause it to follow symlinks instead
of storing them in the zip file.

Relands I4deb98daa9d4ba9f94e3d7670c117fe00381d2ba with tests.

Bug: 112843624
Test: glob_test.go
Test: zip_test.go
Test: m checkbuild
Change-Id: I0eff9c1f2dba79e873fda381ff585df55d5aaaad
</pre>
</div>
</content>
</entry>
<entry>
<title>soong_zip: support globs in -f and -D arguments</title>
<updated>2018-09-28T21:01:29+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-19T00:05:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=1d98ee23a322a997d28ef5dd07412db4b9564d2a'/>
<id>1d98ee23a322a997d28ef5dd07412db4b9564d2a</id>
<content type='text'>
-f and -D arguments can now take globs in the Soong format.

Also update the use of soong_zip that jars resources to escape the
globs in the arguments, and then shell-escape them when writing to
the rsp file so the glob escape are not intepreted by ReadRespFile.

Also remove an unused argument to the buildAAR rule that could
have contained values that needed escaping.

Relands I7f20bb169dc01f952d2a7681ec6ee9c05737ed37 with a fix for
trailing "\n" in list files, which causes a call to pathtools.Glob("")
that returns "./", which could then get incorrectly translated to
"../../../" in the zip file. Also adds tests.

Test: m checkbuild
Test: zip_test.go
Change-Id: I54b8eef9231875e6042a32c9f8bcc5c2f779922a
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
-f and -D arguments can now take globs in the Soong format.

Also update the use of soong_zip that jars resources to escape the
globs in the arguments, and then shell-escape them when writing to
the rsp file so the glob escape are not intepreted by ReadRespFile.

Also remove an unused argument to the buildAAR rule that could
have contained values that needed escaping.

Relands I7f20bb169dc01f952d2a7681ec6ee9c05737ed37 with a fix for
trailing "\n" in list files, which causes a call to pathtools.Glob("")
that returns "./", which could then get incorrectly translated to
"../../../" in the zip file. Also adds tests.

Test: m checkbuild
Test: zip_test.go
Change-Id: I54b8eef9231875e6042a32c9f8bcc5c2f779922a
</pre>
</div>
</content>
</entry>
<entry>
<title>soong_zip: Add tests</title>
<updated>2018-09-28T20:56:06+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-27T22:06:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=05518bc13b3cd31e6cb7158a41aa124b224c3bc4'/>
<id>05518bc13b3cd31e6cb7158a41aa124b224c3bc4</id>
<content type='text'>
Add test that cover basic command line usage of soong_zip.  -D
is not covered yet as the implementation will be replaced with
one that is also more easily testable in the next patch.

Bug: 116751500
Test: zip_test.go
Change-Id: I5a1bcee74ebc9cb3cf332c36f89bc12c0e807ad2
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add test that cover basic command line usage of soong_zip.  -D
is not covered yet as the implementation will be replaced with
one that is also more easily testable in the next patch.

Bug: 116751500
Test: zip_test.go
Change-Id: I5a1bcee74ebc9cb3cf332c36f89bc12c0e807ad2
</pre>
</div>
</content>
</entry>
<entry>
<title>soong_zip: move profiling from zip library to soong_zip</title>
<updated>2018-09-28T20:56:06+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-27T22:04:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=b051ab5cb54f8802807680a8fb24092b445a3ddb'/>
<id>b051ab5cb54f8802807680a8fb24092b445a3ddb</id>
<content type='text'>
Profiling is only used by the standalone soong_zip command,
move it out of the shared zip library.

Bug: 116751500
Test: m checkbuild
Change-Id: I443c34fb39cf8955e163a7720d6f7ed585e4172a
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Profiling is only used by the standalone soong_zip command,
move it out of the shared zip library.

Bug: 116751500
Test: m checkbuild
Change-Id: I443c34fb39cf8955e163a7720d6f7ed585e4172a
</pre>
</div>
</content>
</entry>
<entry>
<title>soong_zip: move args parsing into zip.FileArgsBuilder</title>
<updated>2018-09-28T20:56:06+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2018-09-27T22:00:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/build_soong/commit/?id=fe945b4401a839cc7b177d4d745ab64c8347d93a'/>
<id>fe945b4401a839cc7b177d4d745ab64c8347d93a</id>
<content type='text'>
Add a builder that can construct []FileArg for zip.Run to use.
This will help writing tests that cover the command line parsing.

Test: later cl
Bug: 116751500
Change-Id: I15b5144f5094ab154f13de9c8a84b82db2da4e67
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a builder that can construct []FileArg for zip.Run to use.
This will help writing tests that cover the command line parsing.

Test: later cl
Bug: 116751500
Change-Id: I15b5144f5094ab154f13de9c8a84b82db2da4e67
</pre>
</div>
</content>
</entry>
</feed>
