On Sat, Sep 14, 2024 at 6:38 PM Alex Shi <seakeel@xxxxxxxxx> wrote: > > > On 9/14/24 9:25 AM, Dongliang Mu wrote: > > On Thu, Sep 12, 2024 at 9:22 AM Alex Shi <seakeel@xxxxxxxxx> wrote: > >> > >> > >> On 9/11/24 9:29 PM, Dongliang Mu wrote: > >>> On Wed, Sep 11, 2024 at 6:11 PM Alex Shi <seakeel@xxxxxxxxx> wrote: > >>>> Hi Dongliang: > >>>> > >>>> In scripts/checktransupdate.py, > >>>> def get_origin_path(file_path): > >>>> paths = file_path.split("/") > >>>> tidx = paths.index("translations") > >>>> opaths = paths[:tidx] > >>>> opaths += paths[tidx + 2 :] > >>>> return "/".join(opaths) > >>> cc +ziqiu > >>> > >>> Could you take a look at this function? > >>> > >>>> It use a different way instead of the ':Original:' line to find out the origin file. > >>>> That may cause file finding failed if we have a different dir map for translation. > >>> Yes, this is more reasonable. BTW, is this enforced in the Chinese or > >>> other language translation? > >> It should be used in all language. > > Hi Alex, > > > > Why can't I find this thread in the LKML? We have cc-ed the linux-doc > > mailing list. > > > > Jon has different ideas about this origin file finding. > > Hi Dongliang, > > I just copy to linux-doc, not lkml. but Jon could see the thread in linux-doc too. He reply there may just because more people in LKML. :) > > As to his and others' comments want to do further check, to confirm if the Original file exists. That further checking make sense too, and could be easy to add, isn't it? +1 @Cheng Ziqiu , can you take care the v2 patch which checks the existing of original file instead of using original tag? > > > Thanks > > Alex > > > > >>>> Yes, we have no worry since all of them are same. But could we take the ':Original:' usage for a possible dir map changes? > >>> Yeah, at least we can take the current method as backup for original > >>> file location. > >> good idea! > >> > >>>> Anyway, just a quick idea. > >>>> > >>>> Thanks > >>>> Alex > >>>>