On Wed, Feb 25, 2015 at 08:56:40AM -0500, Patrick Uiterwijk wrote: > ----- Original Message ----- > > On Wed, Feb 25, 2015 at 08:50:25AM -0500, Patrick Uiterwijk wrote: > > > But we could have it fire that same message about the master branch being > > added, > > no? > > (I'm asking if we could move that same piece of code to the place where the > > master branch is being created) > > If we would do that, we would need to also add fedmsg initialization to the setup_git_package script. > While this probably can be done, I think it's most clear to have the python script (pkgdb_sync) initialize it just once and sent the messages there. > Otherwise we'd likely end up with a bunch of json etc in the scripts. It's shell script, so we could just relay on calling fedmsg-logger, the arguments are just the agent, the name of the package and the name of the branch (ie master), 2 of the 3 are hard-coded and we know the third one (the package name). Might not be that much json at the end :) Maybe something along the lines of: fedmsg-logger --message='{"agent": "pkgdb", "name": "$PACKAGE", "branch": "master"}' \ --json-input --modname=git --topic=branch I guess we may want to ask Ralph for some input here But then again, that's something for after the freeze :) Pierre _______________________________________________ infrastructure mailing list infrastructure@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/infrastructure