add uuid option to sipcreator

[raecasey]

ad an arg parse -uuid that allows the user choose to rename the object and related files to a uuid.

[kieranjol]

+1, this should be added ASAP.
Off the top of my head, there already is a uuid option that allows you To specify a uuid for the package rather than getting an auto generated one. So a different argparse name is needed, like rename_uuid or something. Any ideas?

[kieranjol]

This PR has the updates - testing would be ideal in order to ensure that this update is what was required: #314

[raecasey]

thanks for this - i'll test it out in the next two days and let you know

R

[raecasey]

Hi - so I tested this out. am i right in saying that the -rename_uuid argument isn't on the master branch yet?
I got this return on the terminal:
usage: sipcreator.py [-h] -i I [I ...] -o O [-m M] [-u U] [-user USER] [-d]
[-quiet] [-move] [-sc] [-zip] [-oe OE]
[-supplement SUPPLEMENT [SUPPLEMENT ...]]
sipcreator.py: error: unrecognized arguments: -rename_uuid

Do I need to get into a different branch to test it out?

cheers
R

[kieranjol]

Indeed,you must
git checkout rename-uuid

[raecasey]

thanks, - jeez, that was a fast reply.
i typed in
git checkout rename-uuid
git checkout rename_uuid
git checkout rename uuid
but i get this return:
fatal: Not a git repository (or any of the parent directories): .git

i am missing some step. Sorry, - I know you've shown me this loads of times
R

[kieranjol]

No bother,you need to cd Into Ifigit/ifiscripts First,then checkout the branch.

[kieranjol]

Also, GitHub is saying that the pull request can't be merged. There have been alterations to sipcreator since this branch was created so while it's good that you test this out so that it's as requested, the code will have to change prior to merging.

[raecasey]

just tested that out there now, - worked beautifully! thank you

[kieranjol]

Beautiful! Thanks for posting the pic.

[kieranjol]

Hi Raelene,

I resolved the conflict that was preventing the merging. It would be ideal if you could test this out one more time to ensure that the resolved conflict is still creating the desired output.

-K