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

Oops. I was just trying to open a pull for one particular commit. Not all of them. #235

Closed
wants to merge 9 commits into from

Conversation

powerpbx
Copy link

@powerpbx powerpbx commented Mar 7, 2016

No description provided.

@powerpbx
Copy link
Author

powerpbx commented Mar 7, 2016

This is just our first try. If anyone has a better way to do this we we are open to suggestions.

@powerpbx powerpbx closed this Mar 7, 2016
@@ -27,5 +27,5 @@
/**
* The build ID.
*/
public static final String BUILD_ID="0.build.by.SVN";
public static final String BUILD_ID="1";
}
Copy link
Member

Choose a reason for hiding this comment

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

When invoking ant you just need to pass -Dlabel="1" and this is going to be set as build version.

@powerpbx powerpbx changed the title Pipe does not work in windows makefile with Mingw and Msys so we came up with another solution that does. This also removes the last remaining dependency on msys (sed) as far as we can tell. Oops. I was just trying to open a pull for one particular commit. Not all of them. Mar 7, 2016
@ibauersachs
Copy link
Member

You should work on a branch, not on master. And for a pull request you need to do that.

@@ -573,7 +573,7 @@
todir="${light.dir}"
overwrite="true" />

<property name="wix.home" value="C:\Program Files\WiX" />
<property name="wix.home" value="C:\wix310" />
Copy link
Member

Choose a reason for hiding this comment

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

This also can be controlled with passing parameters to ant as -Dwix.home="C:\wix310".

@powerpbx
Copy link
Author

powerpbx commented Mar 7, 2016

Thanks for the info.

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.

None yet

3 participants