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

bug #29 closes , debhelper and standard version updated #35

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Manas-kashyap
Copy link
Contributor

@Manas-kashyap Manas-kashyap commented Dec 24, 2018

Fixes #29
Checklist
I have read the Contribution & Best practices Guide and my PR follows them.
My branch is up-to-date with the Upstream master branch.
I have added necessary documentation (if appropriate)
Added breif description
Changes proposed in this pull request:
Changes in debian/control , and debian/compat for standard version to 4.3.0 (latest released on 23/12/2018) and debhelper 11

Thank you!

@@ -1,3 +1,11 @@
systemlock (0.1-1.1) unstable; urgency=medium

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keep the version same until we implement versioning for this repo

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

means changing it to 0.1-1 right ?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done , please check

@Manas-kashyap Manas-kashyap changed the title bug #29 closes , debhelper and standard version updated bug #29 and #32 closes , debhelper and standard version updated and also correct bug number fixed Dec 26, 2018
Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please solve one issue for one pr
Open up a new branch and then solve bug number issue;

@Manas-kashyap Manas-kashyap changed the title bug #29 and #32 closes , debhelper and standard version updated and also correct bug number fixed bug #29 closes , debhelper and standard version updated Dec 26, 2018
Copy link
Member

@tabesin tabesin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. Pls rebase...

@@ -0,0 +1,13 @@
systemlock (0.1-1) unstable; urgency=medium

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update this file with the correct path

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Work @Manas-kashyap

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much

Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Debhelper 11 is not in the repositories of the travis linux, change it to 9 the build process is failing because of it

Copy link
Contributor Author

@Manas-kashyap Manas-kashyap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doing so

@@ -2,8 +2,8 @@ Source: systemlock
Section: x11
Priority: extra
Maintainer: x-mario <[email protected]>
Build-Depends: debhelper (>= 8.0.0), python(>=2.6), nautilus-admin
Standards-Version: 3.9.2
Build-Depends: debhelper (>= 11), python(>=2.6), gksu

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change it here also

@@ -2,8 +2,8 @@ Source: systemlock
Section: x11
Priority: extra
Maintainer: x-mario <[email protected]>
Build-Depends: debhelper (>= 8.0.0), python(>=2.6), nautilus-admin
Standards-Version: 3.9.2
Build-Depends: debhelper (>= 8.0.0), python(>=2.6), gksu

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't change nautilus-admin here, gksu will not work for the package on latest ubuntu versions their is another pr for it

@@ -1,8 +1,14 @@
systemlock (0.1-1) UNRELEASED; urgency=medium
systemlock (0.1-1.1) UNRELEASED; urgency=medium

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let the version be the same until we implement versioning

Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@meets2tarun @abishekvashok @tabesin please review and merge this pr

Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tabesin this should be merged

Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tabesin this can be merged now

* standard version updated
* debhelper bumped

-- Manas Kashyap <[email protected]> Thu, 03 Jan 2019 21:22:42 +0530
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not properly rebased

Copy link

@pranav1698 pranav1698 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Manas-kashyap can you do the changes as requested by @abishekvashok

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

Successfully merging this pull request may close these issues.

5 participants