dmwm / CRAB2

CRAB2
2 stars 11 forks source link

remove CMSSW version from JDL for gLite #936

Closed belforte closed 10 years ago

belforte commented 10 years ago

we will not publish CMSSW tags any more

chwissing commented 10 years ago

Hi Stefano, during the routine tests by Andrea it turned out that this was not 100% correct. I try ro submit a patch to fix it (and I am completely new to Githup as well).

belforte commented 10 years ago

thanks Christoph, if you tell me what the problem is can also try to fix. I only did one crab -createJdl and verified that CMSSW version was not among the requirements. Or send me the patch and I will commit. I am not sure you can commit to this repository. But have not had the patience to learn git authorization things as well.

chwissing commented 10 years ago

hi Stefano let me try to reply here since I failed to push back my changes (no ideas about Githup authorizations either....).

Find attached a patch for cms_cmssw.py, which should fix it.

I also saw that the version counter is still at 2.9.1, you might want to increase it to 2.9.2 in common.py.

Cheers, Christoph

On 11/22/2013 11:56 AM, belforte wrote:

thanks Christoph, if you tell me what the problem is can also try to fix. I only did one crab -createJdl and verified that CMSSW version was not among the requirements. Or send me the patch and I will commit. I am not sure you can commit to this repository. But have not had the patience to learn git authorization things as well.

— Reply to this email directly or view it on GitHub https://github.com/dmwm/CRAB2/issues/936#issuecomment-29064509.

1022a1023

SB & ChW remove requirement for explict CMSSW version

1024c1025

< req+=' && Member("VO-cms-' + \

        req+='Member("VO-cms-' + \
belforte commented 10 years ago

thanks Christoph, I'll do. stefano

On 11/22/2013 12:09 PM, Christoph Wissing wrote:

hi Stefano let me try to reply here since I failed to push back my changes (no ideas about Githup authorizations either....).

Find attached a patch for cms_cmssw.py, which should fix it.

I also saw that the version counter is still at 2.9.1, you might want to increase it to 2.9.2 in common.py.

Cheers, Christoph

On 11/22/2013 11:56 AM, belforte wrote:

thanks Christoph, if you tell me what the problem is can also try to fix. I only did one crab -createJdl and verified that CMSSW version was not among the requirements. Or send me the patch and I will commit. I am not sure you can commit to this repository. But have not had the patience to learn git authorization things as well.

— Reply to this email directly or view it on GitHub https://github.com/dmwm/CRAB2/issues/936#issuecomment-29064509.

1022a1023

SB & ChW remove requirement for explict CMSSW version

1024c1025

< req+=' && Member("VO-cms-' + \

req+='Member("VO-cms-' + \

— Reply to this email directly or view it on GitHub https://github.com/dmwm/CRAB2/issues/936#issuecomment-29065159.

Stefano Belforte - I.N.F.N. tel : +39 040 375-6261 (fax: 375-6258) Area di Ricerca - Padriciano 99 tel mobile: +39 328 010 7327 34012 TRIESTE TS - Italy AIM: stefanobelforte

belforte commented 10 years ago

took the chance to clean up a bit and make it safe against scram_arch not being defined:

belforte commented 10 years ago

fix released in 2_9_2