Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gathering "package" tags and putting the same "package" tag in several releases? #10

Closed
Martin-Grunewald opened this issue Jun 28, 2013 · 11 comments

Comments

@Martin-Grunewald
Copy link
Contributor

Hi,

I have a question how to most effectively organise the collection of several "package" tags, given that in GIT package tags in the cvs language do not exist as always the full repository is tagged.

In the CVS language, say I have a few new tags of some packages which I want to test together. So I make a release area, and then need to pull in each updated package from someone else's GitHub repository. How does that work if the tags are repository wide? How do I ensure I get from each GiutHub repository only the single/few package I really want?

Related question: in the CVS world it was possible to have a single package tag in several release series (eg, the same HLTrigger/Configuration tag in both 53X and 61X). How does GIT support this, given that tags are not per package? Would I need to develop in one branch and then copy and commit for the other branches, given that "tags" are repository wide?

@Martin-Grunewald
Copy link
Contributor Author

IOW, using git cms-addpkg, is there a way to specify from which (personal) github repository / branch / tag to get the package? BTW, where do I find the documentation of cms-addpkg? The current help text does not say this:

gruen@ughent2:/CMS3/CMSSW_6_2_0_pre8/src/[126]$ git cms-addpkg --help
No manual entry for git-cms-addpkg
gruen@ughent2:
/CMS3/CMSSW_6_2_0_pre8/src/[127]$ git cms-addpkg -h
git cms-addpkg [options] SubSystem/Package

Options:
-h this help message

-d, --debug enable debug output
-f, --force force dangerous operations
--https use https, rather than ssh to access your personal repository
-q, --quiet do not print out progress
-y, --yes assume yes to all questions

@ktf
Copy link
Contributor

ktf commented Jun 28, 2013

No, at the moment, you'll need to use git remote add and git pull to add a different repository and merge your changes. Alternatively you can use git-cms-merge-topic, but that will require the topic being published, kinda like `cmstc tagset -a . I'll comment in a separate thread about the help.

@Martin-Grunewald
Copy link
Contributor Author

Hmm, but how do I specify the individual package I want the updates from, given that I do not want the all repository changes to be pulled in every time?

@davidlt
Copy link
Contributor

davidlt commented Jul 2, 2013

@Martin-Grunewald
Copy link
Contributor Author

As one of the people commented there, this is rather cumbersome (editing config files for each separate case)
and hence can't be put in a script (eg, the use case is that I grab several tags [packages] from different personal
repos). Hence I'd like to ask that cms-addpkg gets empowered to take a repository/branch/tag as an additional argument where to grab things from.... could that be foreseen, please?

@DAMason
Copy link

DAMason commented Jul 11, 2013

Will chime in here since this is relevant to a question I was trying to understand. If this is too old & forgotten I'll open a new issue if nobody sees this :)

So there is the CMSSW development use case which is referred to in the FAQ and here, but there is the probably more common CMSSW analysis use case, where typically an analyzer is pulling in tags from different releases to catch the latest PATtuple recipe, pull in analysis code not tied to a specific release, etc. Quite often we have a dozen-odd packages, each from a different release or standalone tag. To be more specific, how might I do something like:

cvs co -r cms538_v1 -d SusyAnalysis/SusyNtuplizer UserCode/LPCPJM/SusyAnalysis/SusyNtuplizer
cvs co -r V00-00-13 RecoMET/METFilters
cvs co -r V00-00-08 RecoMET/METAnalyzers
cvs co -r V00-11-17 DPGAnalysis/SiStripTools
cvs co -r V00-00-30-01 -d EGamma/EGammaAnalysisTools UserCode/EGamma/EGammaAnalysisTools
cvs co -r V15-01-11 RecoParticleFlow/PFProducer/python
cvs co -r V05-11-02 RecoJets/JetProducers
cvs up -r CMSSW_5_3_8_patch3 RecoJets/JetProducers/interface/JetSpecific.h
cvs up -r CMSSW_5_3_8_patch3 RecoJets/JetProducers/src/JetSpecific.cc
cvs up -r CMSSW_5_3_8_patch3 RecoJets/JetProducers/plugins/CATopJetProducer.h
cvs up -r CMSSW_5_3_8_patch3 RecoJets/JetProducers/plugins/CATopJetProducer.cc
cvs co -r V05-00-16 DataFormats/JetReco
cvs co -r regressionMay18a RecoEgamma/EgammaTools
cvs co -r V00-02-15 DataFormats/StdDictionaries
cvs co -r v1-2-3 -d QuarkGluonTagger/EightTeV UserCode/tomc/QuarkGluonTagger/EightTeV

(The first line is covered now since we moved that guy to GitHub on its own, but the remaining 14 or so lines has me worried...)

Thanks,

--Dave

@davidlt
Copy link
Contributor

davidlt commented Jul 11, 2013

@DAMason, frankly that looks like a mess. It's better to create a branch/repo for your recipe. Instead of doing all this, then I would need just to pull some branch/tag.

@ktf
Copy link
Contributor

ktf commented Jul 15, 2013

As long as your changes are in a release or in the HEAD, you can now follow what described in:

http://cms-sw.github.io/cmssw/advanced-usage.html#i_need_to_checkout_a_file_from_the_old_cvs_head__a_release_tag_how_can_i_do_it

@Martin-Grunewald
Copy link
Contributor Author

Sorry, but concerning the comment by davidlt: "create a branch/repo"
how do you do that? it seems to me that this is what was asked for in the first place,
IOW if we knew how to get a repo with those tags, then the problem is solved (ie, we
would just clone that repo). So how does one make a rep with excatly a specified list of tags
on top of a base release?

@ktf
Copy link
Contributor

ktf commented Aug 9, 2013

@DAMason @Martin-Grunewald can you see if:

#10 (comment)

gives you an idea on how to solve such a usecase in the git way (look at the second part, the first one is for the trivial case).

@Martin-Grunewald
Copy link
Contributor Author

Hi,

I do not understand your reply here: the "#10" link refers to DAMAson's comments further up in the thread,
so how should that be a solution?

yetkinyilmaz referenced this issue in CmsHI/cmssw Dec 23, 2013
kalanand pushed a commit to kalanand/cmssw that referenced this issue Mar 25, 2014
nclopezo pushed a commit to nclopezo/cmssw that referenced this issue Mar 31, 2014
Add changes to the clients which use APVCyclePhaseProducerFromL1TS
nclopezo pushed a commit to nclopezo/cmssw that referenced this issue May 5, 2014
Heavy-ion people assigned to more specific packages
geonmo pushed a commit to geonmo/cmssw that referenced this issue Jul 23, 2014
added npile-up for genericNtuple
shervin86 pushed a commit to shervin86/cmssw that referenced this issue Dec 6, 2014
gpetruc referenced this issue in gpetruc/cmssw Dec 19, 2014
gdimperi pushed a commit to gdimperi/cmssw that referenced this issue Jun 21, 2015
Constructor directly from file
jozzez1 pushed a commit to jozzez1/cmssw that referenced this issue Jul 29, 2015
akalinow pushed a commit to akalinow/cmssw that referenced this issue Sep 14, 2015
slava77 referenced this issue in trackreco/cmssw Jan 13, 2020
vberta referenced this issue in vberta/cmssw Jan 17, 2020
cippy pushed a commit to cippy/cmssw that referenced this issue Mar 14, 2020
Add possibility to save half-precision float variables
lucastorterotot pushed a commit to lucastorterotot/cmssw that referenced this issue Oct 6, 2020
tklijnsma pushed a commit to tklijnsma/cmssw that referenced this issue Nov 19, 2020
…gyCut

Energy threshold on reconstructed candidates
wpmccormack pushed a commit to wpmccormack/cmssw that referenced this issue Aug 17, 2021
cmsbuild pushed a commit that referenced this issue Oct 8, 2021
bendavid pushed a commit to bendavid/cmssw that referenced this issue Feb 15, 2022
Update low pileup recipe to work with high pileup setup
waredjeb pushed a commit to waredjeb/cmssw that referenced this issue Apr 28, 2022
…_12_4_0_pre2

Geometric linking CMSSW_12_4_0_pre2
jsamudio referenced this issue in jsamudio/cmssw Oct 26, 2022
…ackason2_tmp

kernel optimisations. utilize produceLegacy switch.
Pmeiring pushed a commit to Pmeiring/cmssw that referenced this issue Nov 10, 2022
missirol pushed a commit to missirol/cmssw that referenced this issue Feb 23, 2023
…_pre4_stable

Add CaloRecHitSoA and PFRecHitSoA and PFRecHitProducerAlpaka
cmsbuild pushed a commit that referenced this issue Aug 31, 2023
make startup allocations safe on platforms where dlsym allocates memory
jfernan2 pushed a commit to jfernan2/cmssw that referenced this issue Sep 28, 2023
kmohrman pushed a commit to kmohrman/cmssw that referenced this issue Feb 8, 2024
…T_removeExtraTCInfo

Remove extra TC info
mmusich pushed a commit to mmusich/cmssw that referenced this issue Mar 28, 2024
sroychow added a commit to sroychow/cmssw that referenced this issue Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants