create_objectId about bson HOT 12 CLOSED

Hey Arnaud,
Thanks for the bug report. Did you manage to fix your issue ? If so can you send me a pull request ?
If not, I don't have times to fix any issues lately, at best I will try to correct that this upcoming weekend.

Cheers

from bson.

I will fix it tonight

Xinuo

Sent from my iPhone

On 4 Nov 2013, at 21:29, Marc Simon [email protected] wrote:

Hey Arnaud,
Thanks for the bug report. Did you manage to fix your issue ? If so can you send me a pull request ?
If not, I don't have times to fix any issues lately, at best I will try to correct that this upcoming weekend.

Cheers

—
Reply to this email directly or view it on GitHub.

from bson.

The source bson.ml has been fixed and committed. If the value is 24 chars, then should consider it as a hex input, i.e., "99" -> '\153', every two char in the hex string should be converted to one char.

@msimon hope you could help me about other versioning, opam stuff. thanks

from bson.

Sure. No worries. I will do that tomorrow.

Marc Simon
+1 (650)318-1807
Software Engineer
On Nov 5, 2013 12:25 AM, "MassD" [email protected] wrote:

The source bson.ml has been fixed and committed. If the value is 24
chars, then should consider it as a hex input, i.e., "99" -> '\153', every
two char in the hex string should be converted to one char.

@msimon https://github.com/msimon hope you could help me about other
versioning, opam stuff. thanks

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/10#issuecomment-27754282
.

from bson.

@msimon thank you very much.

@ArnaudParant in the mean time, in case you need the fix now, you can pull as the fixed source file has been committed.

from bson.

Hey @MassD @ArnaudParant,
I just update the opam package. It should work, but I didn't test it yet (I don't have ocaml on my work computer....)

from bson.

Hi @MassD @msimon,

Thank you about this quick respond and fix.

I can try the opam package, but what is the source url to add the bson repo in opam ?
$ opam repository add opambson https://github.com/MassD/bson/ ?
This one obviously does not work.

from bson.

Here you go: opam repo add mongo-package [email protected]:MassD/mongo.git

I will try to push it to the official repo very soon.

Cheers

from bson.

should it be opam repo add mongo-package [email protected]:MassD/bson.git?

from bson.

No, I put the opam packages description for bson and mongo in the mongo repo. Not the smartest thing I agree, but it was suppose to be really temporary.

I'll try to fix that asap

from bson.

Thanks Marc.

Best Regards,

Dr. Xinuo Chen

On 5 November 2013 12:47, Marc Simon [email protected] wrote:

No, I put the opam packages description for bson and mongo in the mongo
repo. Not the smartest thing I agree, but it was suppose to be really
temporary.

I'll try to fix that asap

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/10#issuecomment-27769530
.

from bson.

The opam package works correclty.
And the bug is fixed too.

Thank you both again.

from bson.