On 9/15/24 9:20 PM, Dongliang Mu wrote: > 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? As my understanding, Jon want the ':Original:' file, and next he want to check if the file, which get from tag, exist ? :) Thanks > >> >> 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 >>>>>>