On Tue, Sep 11, 2012 at 10:25 PM, Alex Harvey <[email protected]> wrote:

>>
>> I would do that in two commits. Partly because as much as
>> standardizing the naming would be good, it might have to get dropped
>> so as to not break uses of it. Smaller, targeted commits are usually
>> better than larger ones.
>
>
> Okay, I assume the case doesn't actually matter because processor.rb has
> examples of both.  Which is preferred - lower case or camel case?
>

You're right that casing doesn't matter. I thought that you were
talking about changing the name of the fact on openBSD where it seems
to be known as Processor instead of ProcessorCount.

>>
>> I would open another bug or find something that already exists to
>> track adding the HP-UX facts. #11612 seems like it might be closely
>> related.
>
>
> Yes, you are absolutely right; I didn't see that.  That makes it a bit more
> complicated.  I have looked through the code in
> https://github.com/puppetlabs/facter/pull/232 now and I could use much of
> that code and extend it to PA-RISC and the versions of HP-UX I have.  It has
> status "tests insufficient" and no one is assigned, though.  It seems more
> sensible that I continue working with that branch - is that possible?  Can I
> somehow be assigned to the issue, assuming no one is working on it?
>

If it looks like nobody has continued working on it go ahead and
continue with that as a starting point. You can assign the bug to
yourself, I think. If not I can assign it to you.

> --
> You received this message because you are subscribed to the Google Groups
> "Puppet Developers" group.
> To view this discussion on the web visit
> https://groups.google.com/d/msg/puppet-dev/-/cOuDF8BK_DoJ.
>
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected].
> For more options, visit this group at
> http://groups.google.com/group/puppet-dev?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to