Package Details: gn-git r1682.a5bcbd72-1

Git Clone URL: https://aur.archlinux.org/gn-git.git (read-only, click to copy)
Package Base: gn-git
Description: Meta-build system which generates Ninja build files
Upstream URL: https://gn.googlesource.com/gn
Keywords: build chromium gn ninja
Licenses: BSD
Conflicts: gn, gn-bin
Provides: gn
Submitter: aperez
Maintainer: aperez (zopieux)
Last Packager: aperez
Votes: 5
Popularity: 0.000000
First Submitted: 2015-09-14 18:50 (UTC)
Last Updated: 2020-01-02 16:13 (UTC)

Dependencies (4)

Required by (34)

Sources (2)

Latest Comments

« First ‹ Previous 1 2 3 4 Next › Last »

selmf commented on 2018-07-28 12:09 (UTC)

@petronny: That issue was already fixed in the last update the package received.

petronny commented on 2018-07-27 06:19 (UTC)

[234/235] LINK gn
[235/235] LINK gn_unittests
?[1m?[32m==>?(B?[m?[1m Entering fakeroot environment...?(B?[m
?[1m?[32m==>?(B?[m?[1m Starting package()...?(B?[m
install: cannot stat 'gn/tools/gn/docs/*.md': No such file or directory
?[1m?[31m==> ERROR:?(B?[m?[1m A failure occurred in package().?(B?[m

Please fix this.

aperez commented on 2018-07-25 01:35 (UTC)

@selmf: Agreed, I think the “gcc-support.patch“ is small enough that it should not be a problem, so for the moment I would like to keep it. If nobody runs into issues with it (or Gn builds done with it applied and using GCC) maybe I'll submit it upstream later on.

@petronny: That issue should be fixed now.

petronny commented on 2018-07-24 07:43 (UTC)

Cannot find the git package needed to handle git sources. Please fix this.

selmf commented on 2018-07-23 19:35 (UTC) (edited on 2018-07-23 19:36 (UTC) by selmf)

I think we should try to avoid 'overpatching' gen.py. The standalone version of gn is still quite new compared to the old bootstrap method which made previous versions of this package fairly unpredictable. Anything that goes beyond simple adjustments should probably be done (or submitted) upstream to avoid later breakage.

aperez commented on 2018-07-22 17:10 (UTC)

I agree with @selmf that the patch suggested by @bm456 does not seem to make much sense in this case. If something, what I could do is adding some bits to pick the values of CFLAGS/CXXFLAGS/LDFLAGS from the environment, and in that way the settings from “makepkg.conf” would be used for building.

selmf commented on 2018-07-22 16:29 (UTC)

@bm456 What for? That patch removes compiler optimization flags and adds debug symbols. Which will result in a larger binary with worse performance. It also conflicts with the patch to enable gcc builds.

bm456 commented on 2018-07-22 02:54 (UTC)

can u add this patch? 8i've tid to use without i but faint)

thanks https://github.com/gentoo/gentoo/blob/master/dev-util/gn/files/gn-gen-r0.patch

aperez commented on 2018-07-21 20:31 (UTC)

@selfmt Wow, those were really good suggestions, thanks! I have updated the package to take them all into account. Patching the “gen.py” script was easier than expected, so unless upstream changes it a lot making the solution unmaintainable, let's try to keep the patch around.

selmf commented on 2018-07-21 13:47 (UTC)

@aperez, @zopieux: thanks for keeping this updated!

Some suggestions:

  • add --no-sysroot argument when executing gen.py (the script will download a Debian Sid sysroot build environment otherwise)
  • also add --use-lto to take advantage of link time optimization
  • optional: consider building with gcc (This works but it needs CC, CXX and AR environment variables to be set to gcc commads when executing gen.py and you'd need to patch gen.py to remove gcc incompatible flags like '-stdlib=libstdc++' and '-fwhole-program-vtables' - possibly not worth the trouble)