Skip to content
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

Not working for Seapine Surround 2014 version. #43

Open
palash-holkar opened this issue Jun 24, 2016 · 2 comments
Open

Not working for Seapine Surround 2014 version. #43

palash-holkar opened this issue Jun 24, 2016 · 2 comments
Labels

Comments

@palash-holkar
Copy link

Script is not able to parse the output of history command properly.
Also script seems to lack support for handling of whitespaces properly.
I am using 2014.1.0 Build 17.

@JElchison
Copy link
Owner

Hi @palash-holkar! Thanks for your post.

Unfortunately, I haven't tested anything newer than 2013 (see also #42). From the code:

# Last tested using:
#   * Python 2.7.6
#   * GNU bash, version 4.3.11(1)-release (i686-pc-linux-gnu)
#   * sscm command-line client version:  2013.0.0 Build 23 (Linux)
#   * Git version 1.9.1
#   * Ubuntu 14.04.1 LTS
#   * Linux 3.13.0-35-generic #62-Ubuntu SMP Fri Aug 15 01:58:01 UTC 2014 i686 i686 i686 GNU/Linux

There have been a number of forks, so maybe some of those folks have had luck. On my end, and with others that I know, most of us have abandoned hope due to deficiencies in Surround's API and/or CLI. Further, we've learned that some of the issues are technically intractible, like #34.

An example of the API deficiency is the whitespace issue you're citing. If the API were suitable, then there would be no need to parse output from the inconsistent CLI.

Personally, I don't have access to 2014. If you do, I'm more than willing to review your pull requests. Otherwise, I recommend trying your hand with 2013 and the environment cited in the code.

Please report back if you have any better luck!

@palash-holkar
Copy link
Author

palash-holkar commented Jun 27, 2016

Hi @JElchison

Thanks for reply.
I am able to update parsing logic to parse the output of history command(sccm history ...)
But then I got stuck at :
`
print("data %d" % os.path.getsize(localPath))

with open(localPath, "rb") as f:

print(f.read())

return mark
`
Script is opening a file (localPath) and in my case file is .doc file ( & xls too). I guess python can't open doc or xls files, so do you know any alternative to above approach?

Thanks,
Palash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants