Hi,

On Fri, May 31, 2024 at 7:59 PM Gianfranco Costamagna
<locutusofb...@debian.org> wrote:
>
> Well, somebody opened a similar but different approach
> https://salsa.debian.org/ocaml-team/opam/-/merge_requests/9

Sorry, it is me also.:)

But unfortunately, the issue still exists.

It does not work to add this file to debian/clean also.

The issue was raised from when I was trying to debug #1054822 under
chroot, but I never think this will change the file attribute.

Now on salsa repo side:
```
12 -rw-rw-r-- 1 vimer vimer 10160 Apr 21 21:04 appveyor_build.cmd
```
On code from `apt source opam`:
```
12 -rw-r--r-- 1 vimer vimer 10160 May 13  2023 appveyor_build.cmd
```

The odd thing is that I can build it can my local salsa repo. The
tricker I can think of is to change file attribute, but this will
touch upstream code, right?

Sorry for the trouble.

BR,
Bo


>
> G.
>
>
>
>
>
>
> Il venerdì 31 maggio 2024 alle ore 09:09:28 CEST, Bo YU <tsu.y...@gmail.com> 
> ha scritto:
>
>
>
>
>
> Hi Gianfranco!
>
> Thanks for reviewing it.
>
> On Fri, May 31, 2024 at 2:42 PM Gianfranco Costamagna
> <locutusofb...@debian.org> wrote:
> >
> >
> > >    * Exclude these two files
> >
> > Description: appveyor_build.cmd attribute changes due to unknown reason
> > Author: Bo YU <tsu.y...@gmail.com>
> > Last-Update: 2024-04-21
> > ---
> > This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
> > --- a/.gitignore
> > +++ b/.gitignore
> > @@ -1,3 +1,4 @@
> > +appveyor_build.cmd
> >  _build/
> >  _obuild/
> >  _opam/
> >
> >
> > I think you did that to avoid them being autogenerated?
>
> Okay, if this is true then everything becomes easy.
>
> The salsa ci was failed due to the issue:
> https://salsa.debian.org/ocaml-team/opam/-/jobs/5621730
>
> This issue has troubled me for a long time.:(
>
> I will verify this soon.
>
> BR,
>
> Bo
>
> >
> > So in this case, why not just use debian/clean target and add them there?
> >
> > echo appveyor_build.cmd >> debian/clean
> >
> > :)
> >
> > G.
>

Reply via email to