Skip to content

Commit

Permalink
Revert "Merge pull request saltstack#263 from markusgattol/1fa51ff200…
Browse files Browse the repository at this point in the history
…8cc"

This merge was dirty and broke some fixes I just made. And we don't want
to throw scripts in the root dir.
This reverts commit c56fc12, reversing
changes made to 5a8f7ea.
  • Loading branch information
thatch45 committed Nov 25, 2011
1 parent c56fc12 commit fa5c637
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 64 deletions.
27 changes: 0 additions & 27 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -88,30 +88,3 @@ So, please feel free to sprinkle some of this around your systems and
let the deliciousness come forth.

.. _`Apache 2.0 licence`: http://www.apache.org/licenses/LICENSE-2.0.html

Contribute
==========
This world is not salty enough yet... help us make it a saltier place:

* install gitflow e.g. aptitude install git-flow on Debian or from source https://github.com/nvie/gitflow
* fork the salt repository to you your github account
* git clone [email protected]:<youraccount>/salt.git
* cd salt; ./setuprepo.sh

Start contributing... write a test, make sure the test fails, write
the actual code, make the test pass and if it does, make an atomic
commit (referencing the issue(s) if this is a fix) and git push to
your fork. Issue a pull request so one of the saltstack members can
review it and accept or require/advice a change. Lather, rinse,
repeat...

* git pull upstream develop
* write the test, make it fail...
* pep8, pylint, pychecker
* commit, push
* pull request

Following the gitflow branching model is optional i.e. if you prefer a
different branching model then that's fine. All that setuprepo.sh does
is provide you with a quick and hassle-free way to setup the same
branching model that's used for the main repository.
16 changes: 7 additions & 9 deletions salt/master.py
Original file line number Diff line number Diff line change
Expand Up @@ -541,16 +541,14 @@ def run_func(self, func, load):
# Don't honor private functions
if func.startswith('__'):
return self.crypticle.dumps({})
elif func == '_return':
# Don't encrypt the return value for the _return func
# (we don't care about the return value, so why encrypt it?)
# Run the func
ret = getattr(self, func)(load)
# Don't encrypt the return value for the _return func
# (we don't care about the return value, so why encrypt it?)
if func == '_return':
return ret
else:
# Run the func
ret = getattr(self, func)(load)

# AES Encrypt the return
return self.crypticle.dumps(ret)
# AES Encrypt the return
return self.crypticle.dumps(ret)


class ClearFuncs(object):
Expand Down
28 changes: 0 additions & 28 deletions setuprepo.sh

This file was deleted.

0 comments on commit fa5c637

Please sign in to comment.