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

Added command line support for using local CEF builds #44

Merged
merged 1 commit into from
Feb 16, 2017

Conversation

justinstenning
Copy link
Contributor

Example for vs2015 and nupkg-only with local custom build of Cef branch 2924

.\build.ps1 -Target vs2015 -DownloadBinary local -CefVersion 3.2924.1538.gbfdeccd -CefBinaryDir "../../cefsource/chromium/src/cef/binary_distrib/"
.\build.ps1 -Target nupkg-only -DownloadBinary none -CefVersion 3.2924.1538.gbfdeccd

Example for vs2015 and nupkg-only with local custom build of Cef branch 2924
.\build.ps1 -Target vs2015 -DownloadBinary local -CefVersion 3.2924.1538.gbfdeccd -CefBinaryDir "../../cefsource/chromium/src/cef/binary_distrib/"
.\build.ps1 -Target nupkg-only -DownloadBinary none -CefVersion 3.2924.1538.gbfdeccd
@amaitland
Copy link
Member

  • Formatting is a little out, spaces not tabs (Review using the Files Changed tab above).
  • Braces on new lines.

Apart from that looks fine, usage examples are a nice touch 👍

@justinstenning
Copy link
Contributor Author

@amaitland ah yep sorry, edited in the crappy Windows Powershell ISE

@justinstenning
Copy link
Contributor Author

justinstenning commented Feb 16, 2017

@amaitland:
On review - there are no tabs, it is inconsistent spacing that was already there (i.e. it is consistently inconsistent inline with the existing Download function that it is a copy of). Unless I am missing which one you are referring to.

I'm assuming you are referring to the braces in the switch statement (I've changed now to reflect other switch statements. But again it is fairly inconsistent through out.

I'll await further feedback before committing. I'm not going to change any existing functions (which have 2 spaces instead of 4, inconsistent braces), but if you would like me to adjust what I have added to a particular style then please advice correct style (my guess, 4 spaces indent, braces begin on new line).

{
DownloadCefBinaryAndUnzip
"none" {
Copy link
Member

Choose a reason for hiding this comment

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

Brace on new line

sleep -m 2000
#Remove tar file
Remove-Item $TarFile
$Folder = Join-Path $WorkingDir ($Cef64FileName.Substring(0, $Cef64FileName.length - 8))
Copy link
Member

Choose a reason for hiding this comment

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

Indentation is out. Assuming you copied this and the original indentation is out and you have time please fix that also.

{

Write-Diagnostic "Copy $Cef64FileName (approx 200mb)"
Copy-Item ($CefBuildDir+$Cef64FileName) $LocalFile;
Copy link
Member

Choose a reason for hiding this comment

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

Indentation

sleep -m 2000
#Remove tar file
Remove-Item $TarFile
$Folder = Join-Path $WorkingDir ($Cef32FileName.Substring(0, $Cef32FileName.length - 8))
Copy link
Member

Choose a reason for hiding this comment

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

Indentation

@amaitland amaitland merged commit 8253e10 into cefsharp:master Feb 16, 2017
@amaitland
Copy link
Member

Merging now, I'll worry about the formatting later 👍

@justinstenning
Copy link
Contributor Author

Ok no worries, nearly finished, but yeah might be better to do the whole file in one go. Enjoy!

@amaitland
Copy link
Member

but yeah might be better to do the whole file in one go

Yep, it's a mess! Oh well. A problem for another day.

merceyz pushed a commit that referenced this pull request Apr 12, 2017
Added command line support for using local CEF builds

Former-commit-id: 8e8770ee9d7073e25173731400a012d6b44f9eb0
merceyz pushed a commit that referenced this pull request Apr 12, 2017
Added command line support for using local CEF builds

Former-commit-id: 8e8770ee9d7073e25173731400a012d6b44f9eb0
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.

2 participants