From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32309 invoked by alias); 6 Mar 2018 11:28:03 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 32295 invoked by uid 89); 6 Mar 2018 11:28:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=Hx-languages-length:1098 X-HELO: mail-vk0-f67.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=VwsD8LIldSCNaybJJ8UT/Thw2ZfYzzasbwODzhARgBk=; b=e0GTzvcLUIONR6f8M9RxS6rnckhJ/mIvrsKWAkEUzQ+sdIVDodRb52IdT9Vjzve5p3 JuLBLayjvLNcqh058uExPhXG+FT3v3ASzLjP0/J4q4JHzrf31YUNItgn9un8wZ8vsMLH 4YbWjdKqNt4zPfSJJo2gmmtLYjUUJ92vO2/AkA6b8v2mk9Sw0oPbtYuIghyLtNxJRwhU HHem2j+E4LlrVlbr0poS1gUkuJg1SqBY6ETN7H7djR10mpRUwOsjabuOedZRoE9oPIpn XkpM3DR0/cDuQwPZfOEvMpulMCouz3RWBYMZZC7bLhkFfkj+x8VBl/7qxCGQ+j2NNN7Z 1tfA== X-Gm-Message-State: APf1xPAK0nugWg+f1gpFSb4BtM5VTCgfjAEPeuqTpLuLSI6jq4i6BTNw ktJmiuuG5VcG1Ke59uQGEJc8vFF2CSU= X-Google-Smtp-Source: AG47ELt1BX7sxDFyjMr6NwHI3zCaUyj1PI54HAKDiXqR/Wr8jk2v2cHa0IVRqf86/7cbIaPR9D83pA== X-Received: by 10.31.72.130 with SMTP id v124mr13426673vka.144.1520335679544; Tue, 06 Mar 2018 03:27:59 -0800 (PST) Subject: Re: [PATCH 6/7] Consolidate alphasort{64} and versionsort{64} implementation To: Joseph Myers Cc: libc-alpha@sourceware.org References: <1520017165-15830-1-git-send-email-adhemerval.zanella@linaro.org> <1520017165-15830-6-git-send-email-adhemerval.zanella@linaro.org> From: Adhemerval Zanella Message-ID: Date: Tue, 06 Mar 2018 11:28:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2018-03/txt/msg00146.txt.bz2 On 02/03/2018 17:33, Joseph Myers wrote: > On Fri, 2 Mar 2018, Adhemerval Zanella wrote: > >> Also on Linux the compat symbol for old non-LFS dirent64 definition >> requires a platform-specific scandir64.c. For powerpc32 and sparcv9 >> it requires to add specific arch-implementation to override the >> generic Linux one because neither ABI exports an compat symbol for >> non-LFS alphasort64 and versionsort64 variant. Considering both >> architectures do export other compat symbols for other dirent.h >> functions I think this is a bug (I need confirmation). > > If it is a bug, it's also not one that can be fixed (in that there would > be existing binaries expecting both meanings of that symbol at its single > existing version, with binaries expecting the new meaning probably much > more common than those expecting the original meaning of that symbol at > that version). > Right, so I think this initial approach of keep current symbol versioning is the correct one. I will add your remark in commit message.