Skip to content

Factoring visual studio code into a buildgen plugin.#40

Merged
ctiller merged 3 commits intogrpc:masterfrom
nicolasnoble:vsprojects
Jan 23, 2015
Merged

Factoring visual studio code into a buildgen plugin.#40
ctiller merged 3 commits intogrpc:masterfrom
nicolasnoble:vsprojects

Conversation

@nicolasnoble
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor 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
Copy Markdown
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
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Myeah.

Copy link
Copy Markdown
Contributor 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
Copy Markdown
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 [email protected]
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
Copy Markdown
Member

ctiller commented Jan 17, 2015

Is this dead?

@nicolasnoble
Copy link
Copy Markdown
Contributor Author

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

On Fri, Jan 16, 2015 at 5:01 PM, Craig Tiller [email protected]
wrote:

Is this dead?


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

@nicolasnoble
Copy link
Copy Markdown
Contributor 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.

3 participants