New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 719729 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: 1
NextAction: 2017-05-08
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

NFS-Ganesha libntirpc compile errors

Project Member Reported by abiusx@chromium.org, May 8 2017

Issue description

Chrome Version: 60.0.3077.0 unknown (64-bit)
OS: Chromium OS

What steps will reproduce the problem?
(1) Building NFS-ganesha's libntirpc  

What is the expected result?
Build should succeed.

What happens instead?
A couple C warnings/errors prevent completion of the build.

Please use labels and text to provide additional information.
One error in src/authgss_hash.c which uses "abs" over an unsigned int.
Another in src/clnt_vc.c which defines a variable size struct.


 

Comment 1 Deleted

Labels: Proj-Containers
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/d99accc91d1fcc922444ff7f31488ef2c70b0b79

commit d99accc91d1fcc922444ff7f31488ef2c70b0b79
Author: Abbas Naderi <abiusx@chromium.org>
Date: Fri Jul 21 11:09:27 2017

nfs-ganesha: One ebuild to rule them all

nfs-ganesha's build insists on building libntirpc from source, and
also insists on finding that source under its own directory.

nfs-ganesha also needs the libntirpc that is bundled with it, any
other version will result in several symbols remaining unresolved.

The new ebuild pulls both repos (net-fs/nfs-ganesha net-lib/ntirpc)
under one working tree and builds them both. The ebuild for ntirpc
is no longer needed (at least to build ganesha).

BUG= chromium:719729 
TEST=builds succesfully. deployed to device. deployment might have
	dependencies.
CQ-DEPEND=CL:500388

Change-Id: I87f036c43d1be2b3f9a40c7ffde27f769010eb96
Reviewed-on: https://chromium-review.googlesource.com/500748
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Abbas Naderi <abiusx@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/d99accc91d1fcc922444ff7f31488ef2c70b0b79/net-fs/nfs-ganesha/nfs-ganesha-9999.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 21 2017

Labels: merge-merged-V1.3.3-cros
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/ntirpc/+/62f4aac42436e2a7c7662b95c8ffd726444952b3

commit 62f4aac42436e2a7c7662b95c8ffd726444952b3
Author: Abbas Naderi <abisux@google.com>
Date: Fri Jul 21 11:09:26 2017

CHROMIUM: nfs-ganesha: Build errors for libntirpc

Two errors in build, one caused by abs over unsigned int, the other
caused by variable size struct.

BUG= chromium:719729 
TEST=builds succesfully and tested on device
CQ-DEPEND=CL:500748

Change-Id: Ib2ec10d05a1102410454d86a10d133fdaf122d45
Reviewed-on: https://chromium-review.googlesource.com/500388
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Abbas Naderi <abiusx@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>

[modify] https://crrev.com/62f4aac42436e2a7c7662b95c8ffd726444952b3/src/authgss_hash.c
[modify] https://crrev.com/62f4aac42436e2a7c7662b95c8ffd726444952b3/src/clnt_vc.c

Owner: slavamn@chromium.org
please close if stale.
Status: Fixed (was: Started)
No build errors as of today.

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 8 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)
Components: OS>Systems>Containers

Sign in to add a comment