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.

Sign up for GitHub

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

Jump to bottom

INI uses proper line endings #34667

Closed
andras-tim opened this issue Jul 14, 2016 · 0 comments
Closed

INI uses proper line endings #34667

andras-tim opened this issue Jul 14, 2016 · 0 comments

Comments

@andras-tim
Copy link
Contributor

Description of Issue/Question

The ini uses hard-coded \n as line ending. This was so wired.

Setup

call any function from ini

Steps to Reproduce Issue

trivial

Versions Report

Salt Version:
           Salt: 2016.3.1

Dependency Versions:
           cffi: Not Installed
       cherrypy: Not Installed
       dateutil: 1.5
          gitdb: 0.5.4
      gitpython: 0.3.2 RC1
          ioflo: Not Installed
         Jinja2: 2.7.2
        libgit2: Not Installed
        libnacl: Not Installed
       M2Crypto: 0.21.1
           Mako: 0.9.1
   msgpack-pure: Not Installed
 msgpack-python: 0.3.0
   mysql-python: 1.2.3
      pycparser: Not Installed
       pycrypto: 2.6.1
         pygit2: Not Installed
         Python: 2.7.6 (default, Jun 22 2015, 17:58:13)
   python-gnupg: Not Installed
         PyYAML: 3.10
          PyZMQ: 14.0.1
           RAET: Not Installed
          smmap: 0.8.2
        timelib: Not Installed
        Tornado: 4.2.1
            ZMQ: 4.0.4

System Versions:
           dist: Ubuntu 14.04 trusty
        machine: x86_64
        release: 3.13.0-46-generic
         system: Linux
        version: Ubuntu 14.04 trusty
@andras-tim andras-tim mentioned this issue Jul 14, 2016
Merged
cachedout pushed a commit that referenced this issue Jul 14, 2016
Merge pull request #34668 from andras-tim/fix-ini-line-ending
aa4a37a 
INI uses proper line endings - fix #34667
gitebra pushed a commit to gitebra/salt that referenced this issue Jul 15, 2016
@jojohans
Merge remote-tracking branch 'upstream/develop' into develop
1260f2b 
* upstream/develop:
  Fixed file.line on Windows - fix saltstack#34665 (saltstack#34666)
  Revert "Modify lodaer global test to use populated dunders"
  INI uses proper line endings - fix saltstack#34667
  Spelling fixes found in sqlite3 pillar docs (saltstack#34652)
  Lint 34644 (saltstack#34651)
  Adjust the mine test a little bit to give it a better chance of success (saltstack#34647)
  catch error if no dns domains exist
  Check that mysqladmin exists before running mysql integration tests.
  add num_cpus grain to freebsd (saltstack#34565)
  Fixes failing test on Suse Leap (saltstack#34621)
  Changed network state test to use test=True
  Clarifies the proper way to reference states
  Some small changes
  handle jboss cli expression type in the parsing of the output
  Change network state integration test to use test=True
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andras-tim