-
Notifications
You must be signed in to change notification settings - Fork 74
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
"merge" feature doesn't work? #37
Comments
On Wed, 18 Jan 2012, Rubio9 wrote:
Yes, merging does no longer work. The mrepo method is now only used for -- dag wieers, dag@wieers.com, http://dag.wieers.com/ [Any errors in spelling, tact or fact are transmission errors] |
Thanks for clarifying. I assume this is now "working as designed" and won't be changed? |
No, it definitely is not "working as designed", the functionality disappeared when adding functionality that was deemed more important. And the feature never returned. I think the demand is pretty low for this because you can achieve the same thing by simply adding both repositories on the client side. I am open for patches implementing this, however the implementation should only affect the output-part (/var/www/mrepo), not the synchronization part (/var/mrepo). And this implementation could in fact also be merged with the "all" repository. |
Did this ever get fixed? It would be awesome to be able to use this feature |
the documentation indicates:
| ### Example4: Merging repositories
| [dist4]
| name = Distribution example $release ($arch)
| release = 4
| metadata = apt repomd yum
| contrib = rsync://rsync.dist4.org/dist/$release/$arch/$repo/
| extras = rsync://rsync.dist4.org/dist/$release/$arch/$repo/
| merge = mrepo://$nick/contrib/
| mrepo://$nick/extras/
the actual code, however, walks through the listed URLs for a target and, if they are file:// or mrepo://, does a (forced) symlink to the url directory. that means the last url visited is the one and only repository that will be included in the ostensibly merged target. am I doing something wrong?
The text was updated successfully, but these errors were encountered: