alidist icon indicating copy to clipboard operation
alidist copied to clipboard

Move ITSresponse in O2 build_requires

Open mconcas opened this issue 2 years ago • 5 comments

@awegrzyn: FYI

mconcas avatar Dec 13 '22 08:12 mconcas

@mconcas is this package already required for the O2 nightly tag of today?

martenole avatar Dec 13 '22 11:12 martenole

@mconcas is this package already required for the O2 nightly tag of today?

#4701 was merged yesterday, I think so.

mconcas avatar Dec 13 '22 11:12 mconcas

@martenole notice I had to add it to the "override" packages in the Jenkins job description.

ktf avatar Dec 13 '22 16:12 ktf

I was just wondering, because the WeeklyO2Release build uses the most recent FLP suite branch in alidist. But then it takes the o2 recipe from alidist@master. Here then the ITSresponse package is listed, but its not yet included in the alidist branch from the FLP suite. This makes the build fail with ERROR: [Errno 2] No such file or directory: 'alidist/itsresponse.sh' So for building I am now not overwriting the o2 recipe anymore. Of course this would not work if O2 already required it. But it doesn't yet, this I asked Matteo and he confirmed to me on mattermost

martenole avatar Dec 13 '22 16:12 martenole

@martenole You need to add ITSResponse to the ALIDIST_OVERRIDE_PKGS and you should be fine. I did it now for WeeklyO2Release too.

ktf avatar Dec 13 '22 18:12 ktf

@ktf Could you please merge? This breaks our RPM generation.

awegrzyn avatar Dec 15 '22 07:12 awegrzyn