From patchwork Tue Oct 8 20:43:05 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Gabriel F. T. Gomes" X-Patchwork-Id: 1173514 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=sourceware.org (client-ip=209.132.180.131; helo=sourceware.org; envelope-from=libc-alpha-return-105756-incoming=patchwork.ozlabs.org@sourceware.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=inconstante.net.br Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.b="ws0khLOG"; dkim-atps=neutral Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 46nq7Q3Bqcz9sN1 for ; Wed, 9 Oct 2019 07:43:30 +1100 (AEDT) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; q=dns; s=default; b=BignnjooqGx6CF1sQfhnD/Gr15nHOS7pTWaH+UGV3A9 tjMK7TfgaGfn1jNMgjynqzim7OKIKlgZIdJaCFLsved1mOY0q2OWELFNhk7E62sF 6CBvTiE8PWRYU/GQo/0Wly31OtKtURnz6hRMGvCrCvEkCX4CYF9Ba9lZf3PT1BOo = DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-type:content-transfer-encoding; s=default; bh=VBtbhAhSMBNmhzY7M20drO56QJk=; b=ws0khLOGtmFdWGW48 GrtGZZmAkK3CODde4Ez+Bb7dBlhsO8BQkYVF0IArZS8WlEQwBm1ay/PlckLiUjZO Prfe4CXsEnTCLIXa7AA0Fkl3z86T65vvDZWEt6TAREzX71Har1hhy5V3oOn7EabA QtxLq5KGIRL4oWf/41QbAVVAU8= Received: (qmail 58691 invoked by alias); 8 Oct 2019 20:43:24 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 58666 invoked by uid 89); 8 Oct 2019 20:43:24 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-16.9 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.1 spammy=H*F:D*br X-HELO: smtpout1.mo528.mail-out.ovh.net Date: Tue, 8 Oct 2019 17:43:05 -0300 From: "Gabriel F. T. Gomes" To: Florian Weimer CC: , , "Tulio Magno Quites Machado Filho" Subject: [PATCH v2] Make tst-strftime2 depend on locale generation Message-ID: <20191008174305.7763748d@tereshkova> In-Reply-To: <87lftvw12z.fsf@mid.deneb.enyo.de> References: <20191008174509.22788-1-gabriel@inconstante.net.br> <87lftvw12z.fsf@mid.deneb.enyo.de> MIME-Version: 1.0 X-Ovh-Tracer-Id: 17647073665999490651 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgedufedrheelgdduheehucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuqfggjfdpvefjgfevmfevgfenuceurghilhhouhhtmecuhedttdenucesvcftvggtihhpihgvnhhtshculddquddttddm On Tue, 08 Oct 2019, Florian Weimer wrote: > >Doesn't tst-strftime3 have the same problem? Looks good otherwise. On Tue, 08 Oct 2019, Tulio Magno Quites Machado Filho wrote: > >Likewise for tst-strftime3.out ? >I've never seen an error there, but the test does depend on locales too. Yes, you are right. I haven't noticed a problem because these test may get the locale from the system and the locale from the system I'm testing on was good/new enough for tst-strftime3 (but not for tst-strftime2). With the system locales deleted/inaccessible, both tests may fail before v1 of this patch. The following version solves the problem for both tst-strftime2 and tst-strftime3, as well as it adds a ChangeLog entry. Is the following patch OK for master? From 083391d58262b045f39c59253dfb3f60e80a4531 Mon Sep 17 00:00:00 2001 From: "Gabriel F. T. Gomes" Date: Tue, 8 Oct 2019 14:32:17 -0300 Subject: [PATCH v2] Make tst-strftime2 and tst-strftime3 depend on locale generation Building the test cases in parallel might make tst-strftime2 and tst-strftime3 fail. Simply re-running the test case (or building serially) makes the problem go away. This patch adds the necessary dependency to allow parallel builds in the time subdirectory. Tested for powerpc64le. * Makefile: Fix locale dependency for a couple of tests. ($(objpfx)tst-strftime2.out): New rule. ($(objpfx)tst-strftime3.out): Likewise. Reviewed-by: Florian Weimer Reviewed-by: Tulio Magno Quites Machado Filho --- time/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/time/Makefile b/time/Makefile index b9217992a6..ad8844ea34 100644 --- a/time/Makefile +++ b/time/Makefile @@ -61,6 +61,8 @@ include ../gen-locales.mk $(objpfx)tst-ftime_l.out: $(gen-locales) $(objpfx)tst-strptime.out: $(gen-locales) +$(objpfx)tst-strftime2.out: $(gen-locales) +$(objpfx)tst-strftime3.out: $(gen-locales) endif tz-cflags = -DTZDIR='"$(zonedir)"' \