[LU-1965] Lustre --disable-liblustre broken

• Status: Resolved
• Project: Lustre
• Type: Bug
• Priority: Minor
• Reporter: Henri Doreau (Inactive)
• Assignee: Minh Diep
• Resolution: Duplicate
• Labels: build, patch
• Severity: 2

Description

Compilation with --disable-liblustre is broken. According to git bisect, the regression was introduced by commit bfb38a047e0fc12e68a7aa3a2d2ec7a6e9d696ed.

The patch attached, generated against master, seems to fix the issue.

Comments

[Peter Jones]

Henri

Thanks for the patch. Could you please upload the patch into gerrit so that we can review/test/land it? There are instructions relating to this on the Whamcloud wiki and your colleagues should be able to provide guidance on how to set this up if you need it.

Minh

Could you please take care of landing this fix once it is available in gerrit?

Thanks

Peter

[Henri Doreau (Inactive)]

Peter,

thanks, I've pushed my changes to Gerrit. Please let me know if not everything is in order.

http://review.whamcloud.com/#change,4010

[Peter Jones]

Yes - thanks Henri!

[Andreas Dilger]

I think there is already a patch for this, but I can't search for it right now on my phone.

[Andreas Dilger]

This is already being fixed in LU-1905, though it seems that patch hasn't landed yet. A comment in the patch (http://review.whamcloud.com/3953) should help it move along, which I would do now except I'm travelling.

LU-1905 build: Always create libcfs.a

Lustre and LNET utilites have become dependent on libcfs.a but
one can easily disable libcfs.a from being built by disabling
liblustre creation. This patch have Lustre always build libcfs.a.

Signed-off-by: James Simmons <[e-mail]>
Change-Id: Id80f9baa16da238eca708409e0b5d5248e437c5a

Cheers, Andreas
–
Andreas Dilger Whamcloud, Inc.
Principal Lustre Engineer http://www.whamcloud.com/

[Peter Jones]

The fix from LU-1905 has now been landed to master. Please speak up if the problem still remains now.

[Henri Doreau (Inactive)]

The fix from LU-1905 works like a charm.
Sorry for the noise!