Skip to content

[WIP] gitlab: 11.10.4 -> 11.11.3#63056

Closed
mmahut wants to merge 3 commits intoNixOS:masterfrom
mmahut:gitlab
Closed

[WIP] gitlab: 11.10.4 -> 11.11.3#63056
mmahut wants to merge 3 commits intoNixOS:masterfrom
mmahut:gitlab

Conversation

@mmahut
Copy link
Member

@mmahut mmahut commented Jun 13, 2019

Motivation for this change

Major gitlab upgrade 11.10.4 -> 11.11.3.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@mmahut
Copy link
Member Author

mmahut commented Jun 13, 2019

The builds appears to fail which seems possibly an usage of a different ar command?

ar  /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/libs/opt/libares.a /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares__close_sockets.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares__get_hostent.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares__read_line.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares__timeval.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_cancel.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_create_query.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_data.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_destroy.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_expand_name.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_expand_string.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_fds.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_free_hostent.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_free_string.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_getenv.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_gethostbyaddr.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_gethostbyname.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_getnameinfo.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_getopt.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_getsock.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_init.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_library_init.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_llist.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_mkquery.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_nowarn.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_options.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_a_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_aaaa_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_mx_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_naptr_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_ns_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_ptr_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_soa_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_srv_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_parse_txt_reply.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_platform.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_process.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_query.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_search.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_send.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_strcasecmp.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_strdup.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_strerror.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_timeout.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_version.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/ares_writev.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/bitncmp.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/inet_net_pton.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/inet_ntop.o /nix/store/fa2iz4jdcmjhcqqn63kv5mhv1djabby6-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/cares/cares/windows_port.o
ar: invalid option -- '/'

ping @flokli for assistance

@ofborg ofborg bot requested review from fpletz and globin June 13, 2019 06:51
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jun 13, 2019
@flokli
Copy link
Member

flokli commented Jun 13, 2019

Here, it fails in the grpc gem too, but for me while building boringssl, which seems to be vendored inside the grpc gem 😆

I looked into https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/ruby-modules/gem-config/default.nix#L200, so we seem to be able to build a grpc gem successfully, but I'm not sure how to build it properly with bundix.

@alyssais @zimbatm @manveru, any pointers?

@flokli
Copy link
Member

flokli commented Jun 15, 2019

I was mistaken - had a remote builder, which swallowed some of the output. For me, it fails in the ar step, too.

It looks like the rcs part from ar rcs in grpc's Makefile is being thrown away.

@mmahut
Copy link
Member Author

mmahut commented Jun 25, 2019

@alyssais @zimbatm @manveru @flokli any help is welcome :)

@flokli
Copy link
Member

flokli commented Jun 25, 2019

My wild guess would be that the ar line became too long - we could eventually tweak grpc's Makefile to pass relative paths to ar - but as I said, that's just a wild guess.

@bgamari
Copy link
Contributor

bgamari commented Jul 9, 2019

The ar command line looks just plain wrong:

ar  /nix/store/lyack7rns5h345aikzck59l9hsxfxdmn-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/libs/opt/libz.a /nix/store/lyack7rns5h345aikzck59l9hsxfxdmn-ruby2.5.5-grpc-1.19.0/lib/ruby/gems/2.5.0/gems/grpc-1.19.0/src/ruby/ext/grpc/objs/opt/third_party/zlib/adler32.o  ...

It is missing a command character.

It looks to me like the grpc build system expects the AR environment variable to contain not just the path of the ar executable, but also the flags rcs.

@bgamari bgamari mentioned this pull request Jul 10, 2019
10 tasks
@bgamari
Copy link
Contributor

bgamari commented Jul 10, 2019

I have fixed this and opened #64550 bumping gitlab to 12.0.3.

@mmahut
Copy link
Member Author

mmahut commented Jul 10, 2019

Thank you, moving this party to #64550.

@mmahut mmahut closed this Jul 10, 2019
flokli pushed a commit to bgamari/nixpkgs that referenced this pull request Jul 14, 2019
This is a major version bump but things were generally straightforward
save two wrinkles:

 * it is necessary to ignore collisions in the gitlab bundler
   environment as both `omniauth_oauth2_generic` and
   `apollo_upload_server` provide a `console` executable.

 * grpc had to be patched since its build system expects the `AR`
   environment variable to contain not just the path to `ar` but
   also the `rpc` flags (see the discussion in nixpkgs NixOS#63056).
globin pushed a commit to mayflower/nixpkgs that referenced this pull request Jul 16, 2019
This is a major version bump but things were generally straightforward
save two wrinkles:

 * it is necessary to ignore collisions in the gitlab bundler
   environment as both `omniauth_oauth2_generic` and
   `apollo_upload_server` provide a `console` executable.

 * grpc had to be patched since its build system expects the `AR`
   environment variable to contain not just the path to `ar` but
   also the `rpc` flags (see the discussion in nixpkgs NixOS#63056).
flokli pushed a commit to flokli/nixpkgs that referenced this pull request Aug 20, 2019
This is a major version bump but things were generally straightforward
save two wrinkles:

 * it is necessary to ignore collisions in the gitlab bundler
   environment as both `omniauth_oauth2_generic` and
   `apollo_upload_server` provide a `console` executable.

 * grpc had to be patched since its build system expects the `AR`
   environment variable to contain not just the path to `ar` but
   also the `rpc` flags (see the discussion in nixpkgs NixOS#63056).

(cherry picked from commit 363b352)
ciil pushed a commit to mayflower/nixpkgs that referenced this pull request Aug 21, 2019
This is a major version bump but things were generally straightforward
save two wrinkles:

 * it is necessary to ignore collisions in the gitlab bundler
   environment as both `omniauth_oauth2_generic` and
   `apollo_upload_server` provide a `console` executable.

 * grpc had to be patched since its build system expects the `AR`
   environment variable to contain not just the path to `ar` but
   also the `rpc` flags (see the discussion in nixpkgs NixOS#63056).

(cherry picked from commit 363b352)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants