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

Factoring visual studio code into a buildgen plugin. #40

Merged
merged 3 commits into from
Jan 23, 2015
Merged

Factoring visual studio code into a buildgen plugin. #40

merged 3 commits into from
Jan 23, 2015

Conversation

nicolasnoble
Copy link
Member

No description provided.

projects = []
projects.extend(libs)
projects.extend(targets)
projects = [project for project in projects if project.get('vs_project_guid', None)]
Copy link
Member

Choose a reason for hiding this comment

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

Line length

Copy link
Member

Choose a reason for hiding this comment

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

This needs a comment - what is it doing

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question - I don't know :-D

This is me moving and de-duplicating code from @jtattermusch at that point. But I too was wondering what that was.

Copy link
Member

Choose a reason for hiding this comment

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

Reverse engineering:

"select projects that have a vs_project_guid tag set"

Copy link
Member Author

Choose a reason for hiding this comment

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

Myeah.

Copy link
Member Author

Choose a reason for hiding this comment

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

Figured it out actually. Tests don't have a guid because we don't want to create a project file for them.

Copy link
Contributor

Choose a reason for hiding this comment

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

On Thu, Jan 22, 2015 at 5:25 PM, Nicolas Noble notifications@github.com
wrote:

In tools/buildgen/plugins/generate_vsprojects.py
#40 (comment):

  • We want to help the work of the visual studio generators.
  • """
  • libs = dictionary.get('libs', [])
  • targets = dictionary.get('targets', [])
  • for lib in libs:
  • lib['is_library'] = True
  • for target in targets:
  • target['is_library'] = False
  • projects = []
  • projects.extend(libs)
  • projects.extend(targets)
  • projects = [project for project in projects if project.get('vs_project_guid', None)]

Figured it out actually. Tests don't have a guid because we don't want to
create a project file for them.

Oh, yes that's the point. Sorry I didn't react to this right away.


Reply to this email directly or view it on GitHub
https://github.com/google/grpc/pull/40/files#r23426008.

@ctiller
Copy link
Member

ctiller commented Jan 17, 2015

Is this dead?

@nicolasnoble
Copy link
Member Author

No. Will do this during the week-end :-P

On Fri, Jan 16, 2015 at 5:01 PM, Craig Tiller notifications@github.com
wrote:

Is this dead?


Reply to this email directly or view it on GitHub
#40 (comment).

@nicolasnoble
Copy link
Member Author

Done.

ctiller added a commit that referenced this pull request Jan 23, 2015
Factoring visual studio code into a buildgen plugin.
@ctiller ctiller merged commit e117547 into grpc:master Jan 23, 2015
stanley-cheung referenced this pull request in stanley-cheung/grpc Aug 27, 2015
Add a dummy implementation of route_guide in c++.
apolcyn referenced this pull request in apolcyn/grpc Jul 25, 2016
Add PHP example to public auth doc
jtattermusch referenced this pull request in jtattermusch/grpc Dec 13, 2016
Make Node extension work with slice changes
lalo pushed a commit to lalo/grpc that referenced this pull request Oct 19, 2017
Fix bug in JSON parsing when string value is a date
@lock lock bot locked as resolved and limited conversation to collaborators Feb 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants