2.0.12 bug fixes#114
Conversation
|
I guess I didn't quite follow the preferred work flow here. I should have started one new branch (in my repo) per bug fix, named appropriately, and PR'd them separately. I'll try to do a better job in the future :) -Glen |
|
No problem :) |
|
I'm a little confused on the version numbers I think :( Shay mentioned using the third digit for patch level. If we go that route, We're scrapping 2.0.13 (which was not going to be just a patch to 2.0.12) -Glen On Wed, Dec 4, 2013 at 8:08 AM, Francisco Figueiredo Jr. <
|
|
I can understand the confusion :) As background, try reading http://semver.org, which is a very popular To answer your specific question(s)... in the semver world, as long as we What's maybe a bit confusing (I just realized this myself), is that we'd PS One think I'm not sure I like with semver is that any API breaking Am 4 Dec 2013 um 19:35 schrieb Glen Parker notifications@github.com: I'm a little confused on the version numbers I think :( Shay mentioned using the third digit for patch level. If we go that route, We're scrapping 2.0.13 (which was not going to be just a patch to 2.0.12) -Glen On Wed, Dec 4, 2013 at 8:08 AM, Francisco Figueiredo Jr. <
— |
|
On Wed, Dec 4, 2013 at 10:31 AM, Shay Rojansky notifications@github.comwrote:
OK, thanks for clarifying. Makes more sense now. Although I still think we need to get a patch for 2.0.12 out ASAP, because Would it be better to release a 2.0.12.1, as kind of a special case? Or go
I agree, it's a really nice guideline, and like all guidelines, a little -Glen |
|
On Wed, Dec 4, 2013 at 8:58 PM, Glen Parker notifications@github.comwrote:
Sure, you definitely have a point. When the discussion started I was under It makes perfect sense to release a bugfix because of the 9.3 situation,
Yep :)
|
Well I branched from 2.0.12. If we call that 2.0.13, then 2.0.13 will not So you're saying to abandon 2.0.12 and branch at the v2.0.13-beta1 tag, I'll branch at 2.0.13, fix it, and we can test it out :) -Glen |
And there it is. -Glen |
|
On Wed, Dec 4, 2013 at 9:21 PM, Glen Parker notifications@github.comwrote:
Just looked at the state of things in git a little. I think I misunderstood If I understand correctly, on June 27 we tagged the master as 2.0.13-beta1 Since 2.0.13-beta1 isn't light at all (many changes since 2.0.12, I say go for it :) You don't even need to create a branch, just checkout Am I finally understanding everything? So you're saying to abandon 2.0.12 and branch at the v2.0.13-beta1 tag,
|
|
On Wed, Dec 4, 2013 at 11:40 AM, Shay Rojansky notifications@github.comwrote:
Yep, sound like you've got it :) So, 2.0.12 and 2.0.13 are branched and fixed now. If we jump 2.0.12 to But that's about where I jump off the bus; I have no idea how to go about -Glen |
|
@franciscojunior, I hope we didn't drive you crazy here :) Glen thinks (and I agree after finally understanding the whole situation) that we should quickly release a 2.0.14 version, which is basically 2.0.12 + the urgent NpgsqlError bug that's causing people a lot of trouble. Then, we can calmly prepare the 2.1.0 release (@glenebob, I don't see the 2.0.14 tag yet, can you create it in npgsql/npgsql? Do you agree that this is a good idea? If so, can you do the release with the current manual nuget flow (for 2.1.0 we can use the build server and everything). |
|
On Wed, Dec 4, 2013 at 11:55 AM, Shay Rojansky notifications@github.comwrote:
OK, I updated the release files in the 2.0.12 branch and tagged v2.0.14. -Glen
|
|
I say +1 for it. I know users will get a little bit confused by the fact that a 2.0.14 has About the release process, Glen, we need to create zip files with versions
|
|
On Wed, Dec 4, 2013 at 5:53 PM, Francisco Figueiredo Jr. <
Sure, count me in! Now that the build server is up and running, will it be useful for this old -Glen |
|
2.0.14 doesn't include many changes I did to make the build server process work /be streamlined... My suggestion is that since you want 2.0.14 out as fast as possible, just go ahead and create the packages the old way. We'll start using the build server for 2.1.0... |
|
That's what I figured. I played around a bit earlier tonight and ran into -Glen On Wed, Dec 4, 2013 at 10:34 PM, Shay Rojansky notifications@github.comwrote:
|
|
Which problems did you get, Glen? On Thu, Dec 5, 2013 at 4:56 AM, Glen Parker notifications@github.comwrote:
Regards, Francisco Figueiredo Jr. |
|
The build scripts don't work. Build.bat says no .net SDK installed, which What exactly is the "normal" procedure for building Npgsql for release? -Glen On Thu, Dec 5, 2013 at 3:22 AM, Francisco Figueiredo Jr. <
|
Build would call nant to create the Npgsql.dll file. But I think Josh Cooley didn't use the build.bat file. I think he used vs.net directly to create the assemblies. I just sent him a mail talking about this discussion and asking him how he created the windows packages. The policyFileBuild script has to be run in the vs.net command prompt. This is because vs.net command prompt sets some env variables which the script uses.
I also asked Josh which day would be better for him to participate in the creation of the 2.0.14 release. He said that Saturday, Sunday or Monday after 8 pm is ok. Or Tuesday or Thursday next week. |
|
Guys, I suggest that for 2.0.14 you do it in "quick and dirty"mode :) This is probably the last release we'll be doing without the new build system, build server, etc. which will make all this much much easier. Also, don't forget to make a nuget package - I think this is how most people nowadays use npgsql. You can take the nuspec from the master and modify it accordingly, and then run nuget -pack on it. If you want/need help from me with any of this let me know. |
|
Yes, I did the builds in visual studio. I probably should have created an On Fri, Dec 6, 2013 at 5:20 AM, Francisco Figueiredo Jr. <
|
|
@jbcooley, I just noticed npgsql has a publisher policy. I'm not super-familiar with this feature, and wanted to understand a bit more...
Basically I haven't seen this in other open source projects and am trying to understand the idea. If we keep it I'll integrate it into our new build process too... |
|
I think we started to use this policy file because we had some reports of users who installed a newer version and had their applications not working anymore. This was because they were compiling the app against an specific Npgsql version and after the update the linking couldn't be satisfied anymore. |
|
Hmm, ok. I don't really know so much about these "redirection" mechanisms. I think maybe this should in general be more under the control of the user, and not via a "publisher-pushed" policy. Users can do many things, they can:
Basically this seems to be more of a user-side problem, which we shouldn't interfere with (using something like publisher policies). But I'm really not very sure, can someone else give their opinion? |
|
The two other providers I'm familiar with using (for SQL Server, and There are two ways to deploy Npgsql. One way is to deploy it to the GAC One scenario where the publisher policy file can be extremely useful is Ultimately, I believe my stance is very close to yours. We shouldn't On Fri, Dec 6, 2013 at 12:20 PM, Shay Rojansky notifications@github.comwrote:
|
|
OK guys, I think I have the binary and source .zip files ready to go. I added VS2012 files to git and used them to do the builds. I've got I have no idea how to generate the api-docs, or the nuget package. Francisco, if you want to me to upload the files directly to pg-foundry, -Glen On Fri, Dec 6, 2013 at 11:21 AM, jbcooley notifications@github.com wrote:
|
|
On Fri, Dec 6, 2013 at 7:52 AM, jbcooley notifications@github.com wrote:
That's what I did. I've been having trouble with VS though. When I click -Glen |
|
On Fri, Dec 6, 2013 at 3:20 AM, Francisco Figueiredo Jr. <
-Glen |
|
@glenebob, for the nuget package look at this as an example: https://github.com/npgsql/Npgsql/blob/master/Npgsql/Npgsql.nuspec It just needs to point to the actual DLLs (and possibly xmls) you want to package. No need for the .pdb and .cs files (as you won't be building a symbol/debug nuget). Don't forget to update the version though. |
|
XML documentation can be generated with the compiler's /doc: feature: http://msdn.microsoft.com/en-us/library/3260k4x7.aspx |
|
@jbcooley, regarding the publisher policy, I see what you mean. The whole discussion seems to be a part of the mode where users install npgsql into their GAC (publisher policies don't make any sense otherwise, right?). This seems to be an installation flow that's completely separate from nuget. It seems to make sense to continue to distribute a zip for those cases, which would contain the publisher policy (which the user can choose to install into the GAC or not). One additional thing to think about - it seems we're leaning towards more semver-like versioning. This might mean we do publisher policies only within the same minor version, but not across minor versions. In other words, publisher policies would specify equivalence from 2.0.x to the latest 2.0 version (i.e. 2.0.14), but not to 2.1. This is since 2.0->2.1 represents a much heavier upgrade which is a less likely to be 100% smooth. Does that sound right? In any case, I'll start looking at automating the zip packaging on the build server for 2.1.0 and on. |
|
The XML documentation isn't the issue - it's getting generated just fine. What I'm referring to is the html apidocs as found here... -Glen On Fri, Dec 6, 2013 at 1:53 PM, Shay Rojansky notifications@github.comwrote:
|
|
I used nant to generate the com. I think it can also generate the html docs. Josh On Dec 6, 2013, at 7:27 PM, Glen Parker notifications@github.com wrote: The XML documentation isn't the issue - it's getting generated just fine. What I'm referring to is the html apidocs as found here... -Glen On Fri, Dec 6, 2013 at 1:53 PM, Shay Rojansky notifications@github.comwrote:
— |
|
I that sounds right. I know there's been discussion on dropping v2 Josh On Dec 6, 2013, at 4:31 PM, Shay Rojansky notifications@github.com wrote: @jbcooley https://github.com/jbcooley, regarding the publisher policy, I The whole discussion seems to be a part of the mode where users install One additional thing to think about - it seems we're leaning towards more In any case, I'll start looking at automating the zip packaging on the — |
|
Em 06/12/2013 23:27, "Glen Parker" notifications@github.com escreveu:
Those are created with ./build.sh apidocs ( or build.bat if you are running I create those when creating the mono release.
|
|
Em 06/12/2013 19:20, "Glen Parker" notifications@github.com escreveu:
Excellent, Glen!
Sure! I'll add your name to the project. About your username, how did you
Sure. I'd like to check with you if we could try to make the release I think we all can check if everything is OK and make the release. What do you think?
|
|
Em 06/12/2013 19:27, "Glen Parker" notifications@github.com escreveu:
Great! So I think a release tonight may be possible. He is also from USA. His time zone is gmt - 6. There is no problem if something comes up and we can't meet to make the I also think we can make the release asynchronously. Once the reason can be
|
|
Glen, when I tried to add you to pgfoundry, the system said the user name
|
|
On Sat, Dec 7, 2013 at 2:51 AM, Francisco Figueiredo Jr. <
OK. I stole some of Shay's most excellent project file magic and merged it
-Glen |
|
On Sat, Dec 7, 2013 at 3:17 AM, Francisco Figueiredo Jr. <
-Glen |
Francisco,
I deleted the v2.0.12.1 branch I created earlier, and created a new one that matches the naming convention of 2.1.
So now I'm presenting the bug fixes in that branch as PR's instead.
-Glen