Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-2302

Use apt instead of yum in setup-env-debian.sh

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: None
    • Labels:
      None

      Description

      The file bigtop-deploy/vm/utils/setup-env-debian.sh uses yum instead of apt when discussing the local repo.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user fsareshwala opened a pull request:

        https://github.com/apache/bigtop/pull/78

        BIGTOP-2302. Use apt instead of yum in setup-env-debian.sh

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/fsareshwala/bigtop bugfix/setup-env

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/bigtop/pull/78.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #78


        commit 942f6a79c25d0db5b6ff3ed024f4771b011a4ea0
        Author: Faraaz Sareshwala <fsareshwala@quantcast.com>
        Date: 2016-02-02T01:21:11Z

        BIGTOP-2302. Use apt instead of yum in setup-env-debian.sh


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user fsareshwala opened a pull request: https://github.com/apache/bigtop/pull/78 BIGTOP-2302 . Use apt instead of yum in setup-env-debian.sh You can merge this pull request into a Git repository by running: $ git pull https://github.com/fsareshwala/bigtop bugfix/setup-env Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bigtop/pull/78.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #78 commit 942f6a79c25d0db5b6ff3ed024f4771b011a4ea0 Author: Faraaz Sareshwala <fsareshwala@quantcast.com> Date: 2016-02-02T01:21:11Z BIGTOP-2302 . Use apt instead of yum in setup-env-debian.sh
        Hide
        oflebbe Olaf Flebbe added a comment -

        LGTM, +1

        Show
        oflebbe Olaf Flebbe added a comment - LGTM, +1
        Hide
        oflebbe Olaf Flebbe added a comment -

        Committed, thanks!

        Show
        oflebbe Olaf Flebbe added a comment - Committed, thanks!
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user fsareshwala closed the pull request at:

        https://github.com/apache/bigtop/pull/78

        Show
        githubbot ASF GitHub Bot added a comment - Github user fsareshwala closed the pull request at: https://github.com/apache/bigtop/pull/78
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user fsareshwala commented on the pull request:

        https://github.com/apache/bigtop/pull/78#issuecomment-178812854

        This was committed.

        https://issues.apache.org/jira/browse/BIGTOP-2302

        Show
        githubbot ASF GitHub Bot added a comment - Github user fsareshwala commented on the pull request: https://github.com/apache/bigtop/pull/78#issuecomment-178812854 This was committed. https://issues.apache.org/jira/browse/BIGTOP-2302
        Hide
        cos Konstantin Boudnik added a comment -

        The JIRA title and commit message differs. For the patch author: please make sure they match, in the future. Thanks!

        Show
        cos Konstantin Boudnik added a comment - The JIRA title and commit message differs. For the patch author: please make sure they match, in the future. Thanks!
        Hide
        cos Konstantin Boudnik added a comment -

        I've fixed the JIRA title, as I can not amend the commit

        Show
        cos Konstantin Boudnik added a comment - I've fixed the JIRA title, as I can not amend the commit
        Hide
        fsareshwala Faraaz Sareshwala added a comment -

        Thank you! I'll be sure to keep them in sync in the future.

        Show
        fsareshwala Faraaz Sareshwala added a comment - Thank you! I'll be sure to keep them in sync in the future.

          People

          • Assignee:
            fsareshwala Faraaz Sareshwala
            Reporter:
            fsareshwala Faraaz Sareshwala
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development