On Thu, Aug 05, 2010 at 03:50:58PM -0700, Brock Pytlik wrote: > On 08/ 5/10 01:20 PM, [email protected] wrote: > >On the depot side, I have some additional concerns about file/1. In > >particular, we enable certain operations based upon what mode the depot > >is in. There's a map for everything, a map for read-only, and a map for > >mirror mode. In the case of POST file/1, it looks like this operation > >is enabled even if the depot is in readonly or mirror mode. At a > >minimum, we need to make sure that you can't POST a file in these modes. > >I'm also a bit concerned about how we determine whether POST for file/1 > >is enabled by the client. Up to this point, the versions/0 information > >tells us whether the depot supports a particular operation, and what > >versions are supported. If a depot isn't providing a particular > >operation, the operation string is absent in the versions/0 information. > >However, with file/1, we can support GET file/1, but not POST file/1. > >The versions/0 information doesn't currently have a way of expressing > >this. Shawn may not agree with me, but perhaps add_cert, or whatever > >you had before, is actually preferable? Thoughts? > > I see both sides of this and I don't really have a strong opinion > either way about what the right solution is. I've been following the > discussion between you and Shawn. What I'd like is for you two to > either come to a consensus and let me know which is preferred (so > that I don't have to change it back if I change it again), or decide > that there won't be consensus between you two and then I'll just > decide.
That's reasonable. I'm not trying to come up with additional issues to stall your progress. It's just that these keep popping up while I'm working on other parts of the code. I'm assuming that's because we haven't finished designing the new publication interface. We've brainstormed about it (and come up with some neat ideas), but nobody has written up a document, and committed it to a review. I'm not really sure that it's fair to attach requirements to your project for interfaces that are mostly still in our heads. -j _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
