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

Remove gflags dependancy #165

Closed
tresni opened this issue Dec 10, 2014 · 6 comments
Closed

Remove gflags dependancy #165

tresni opened this issue Dec 10, 2014 · 6 comments
Assignees
Milestone

Comments

@tresni
Copy link
Collaborator

tresni commented Dec 10, 2014

gflags has not been maintained for 2 years. It works, but Google has actually moved all their python projects back to argparse. We should look at doing the same.

@tresni tresni self-assigned this Dec 10, 2014
@tresni tresni added this to the 3.4 milestone Dec 10, 2014
@tresni
Copy link
Collaborator Author

tresni commented Feb 5, 2015

Note to self: Always push local branches before you do a clean install of OS... So much work, gone.. 😢

@insanum
Copy link
Owner

insanum commented Feb 13, 2015

I feel your pain and I've been there before. This led me to start using unison. It's awesome as it runs over ssh via a cron job. I sync multiple systems this way and never had a problem since. Love it.

@tresni tresni mentioned this issue Jun 11, 2015
tresni added a commit that referenced this issue Sep 8, 2015
Just a point release to address the issue for now.
Replacing argparse is already covered in #165 for 3.4
@tresni
Copy link
Collaborator Author

tresni commented Sep 8, 2015

For anyone who wants to see where this is going: https://github.com/insanum/gcalcli/compare/argparse_over_gflags

Notable Changes:

  • removed --flagfile=<file> in favor of argparse @<file> syntax
  • removed the --detail_* flags in favor of --detail [all|calendar|location|url|...]

Lots of TODO's still...

@polyzen
Copy link

polyzen commented Oct 2, 2015

Is this ready for consumption? :)

@tresni
Copy link
Collaborator Author

tresni commented Oct 3, 2015

Yes-ish. It means that there are a lot of behavior changes in how we parse command line options that I haven't documented yet, but gcalcli --help will likely get you the info you need. I have been using it and tried to test all the angles, but I may have missed things. So think of it as alpha ready.

Sent from my iPhone

On Oct 2, 2015, at 7:24 PM, Daniel M. Capella notifications@github.com wrote:

Is this ready for consumption? :)


Reply to this email directly or view it on GitHub.

uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 30, 2016
gcalcli requires a dependency on oauth2client <= 1.4.12 to work
correctly, as higher versions replaced the run() method with run_flow().

The version of oauth2client in ports is 2.2.0 at the time of writing.

Fixes for the issue [1][2][3] have been submitted upstream in multiple
cases, although upstream has closed them as duplicates of a refactor
issue [4] that is yet to be committed/resolved.

This commit commits the proposed change in issue #229 [2].

While I'm here:

* Explicitly specify (and limit) supported Python versions to -2.x
* Add python to CATEGORIES
* Group, re-order and sort USE{s}, NO_* and files/plist sections
* Capitalize OPTIONS descriptions

[1] insanum/gcalcli#211
[2] insanum/gcalcli#229
[3] insanum/gcalcli#244
[4] insanum/gcalcli#165

PR:		206045
Submitted by:	Lawrence Chen <beastie tardisi com>
Approved by:	portmgr (maintainer timeout, 6+ months)
MFH:		2016Q3


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@419306 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 30, 2016
gcalcli requires a dependency on oauth2client <= 1.4.12 to work
correctly, as higher versions replaced the run() method with run_flow().

The version of oauth2client in ports is 2.2.0 at the time of writing.

Fixes for the issue [1][2][3] have been submitted upstream in multiple
cases, although upstream has closed them as duplicates of a refactor
issue [4] that is yet to be committed/resolved.

This commit commits the proposed change in issue #229 [2].

While I'm here:

* Explicitly specify (and limit) supported Python versions to -2.x
* Add python to CATEGORIES
* Group, re-order and sort USE{s}, NO_* and files/plist sections
* Capitalize OPTIONS descriptions

[1] insanum/gcalcli#211
[2] insanum/gcalcli#229
[3] insanum/gcalcli#244
[4] insanum/gcalcli#165

PR:		206045
Submitted by:	Lawrence Chen <beastie tardisi com>
Approved by:	portmgr (maintainer timeout, 6+ months)
MFH:		2016Q3
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Jul 30, 2016
gcalcli requires a dependency on oauth2client <= 1.4.12 to work
correctly, as higher versions replaced the run() method with run_flow().

The version of oauth2client in ports is 2.2.0 at the time of writing.

Fixes for the issue [1][2][3] have been submitted upstream in multiple
cases, although upstream has closed them as duplicates of a refactor
issue [4] that is yet to be committed/resolved.

This commit commits the proposed change in issue #229 [2].

While I'm here:

* Explicitly specify (and limit) supported Python versions to -2.x
* Add python to CATEGORIES
* Group, re-order and sort USE{s}, NO_* and files/plist sections
* Capitalize OPTIONS descriptions

[1] insanum/gcalcli#211
[2] insanum/gcalcli#229
[3] insanum/gcalcli#244
[4] insanum/gcalcli#165

PR:		206045
Submitted by:	Lawrence Chen <beastie tardisi com>
Approved by:	portmgr (maintainer timeout, 6+ months)

Approved by:	ports-secteam (blanket)
@tresni
Copy link
Collaborator Author

tresni commented Aug 26, 2016

This has been done. See instructions in #237 (comment)

@tresni tresni closed this as completed Aug 26, 2016
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Dec 2, 2016
gcalcli requires a dependency on oauth2client <= 1.4.12 to work
correctly, as higher versions replaced the run() method with run_flow().

The version of oauth2client in ports is 2.2.0 at the time of writing.

Fixes for the issue [1][2][3] have been submitted upstream in multiple
cases, although upstream has closed them as duplicates of a refactor
issue [4] that is yet to be committed/resolved.

This commit commits the proposed change in issue #229 [2].

While I'm here:

* Explicitly specify (and limit) supported Python versions to -2.x
* Add python to CATEGORIES
* Group, re-order and sort USE{s}, NO_* and files/plist sections
* Capitalize OPTIONS descriptions

[1] insanum/gcalcli#211
[2] insanum/gcalcli#229
[3] insanum/gcalcli#244
[4] insanum/gcalcli#165

PR:		206045
Submitted by:	Lawrence Chen <beastie tardisi com>
Approved by:	portmgr (maintainer timeout, 6+ months)

Approved by:	ports-secteam (blanket)
@jcrowgey jcrowgey modified the milestones: 4.1, 4.0 Sep 7, 2018
uqs pushed a commit to freebsd/freebsd-ports that referenced this issue Apr 1, 2021
gcalcli requires a dependency on oauth2client <= 1.4.12 to work
correctly, as higher versions replaced the run() method with run_flow().

The version of oauth2client in ports is 2.2.0 at the time of writing.

Fixes for the issue [1][2][3] have been submitted upstream in multiple
cases, although upstream has closed them as duplicates of a refactor
issue [4] that is yet to be committed/resolved.

This commit commits the proposed change in issue #229 [2].

While I'm here:

* Explicitly specify (and limit) supported Python versions to -2.x
* Add python to CATEGORIES
* Group, re-order and sort USE{s}, NO_* and files/plist sections
* Capitalize OPTIONS descriptions

[1] insanum/gcalcli#211
[2] insanum/gcalcli#229
[3] insanum/gcalcli#244
[4] insanum/gcalcli#165

PR:		206045
Submitted by:	Lawrence Chen <beastie tardisi com>
Approved by:	portmgr (maintainer timeout, 6+ months)

Approved by:	ports-secteam (blanket)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants