Hi team, I created bug 518410 so everyone can vote if we should revert or not the patches for bug 515526<https://bugs.eclipse.org/bugs/show_bug.cgi?id=515526> and bug 518128<https://bugs.eclipse.org/bugs/show_bug.cgi?id=518128> in PDT 5.0.1
Please have a look at https://bugs.eclipse.org/bugs/show_bug.cgi?id=518410 when you have some time, your comments and votes will be very highly appreciated :) Have a nice week-end, Thierry. ________________________________ De : pdt-dev-boun...@eclipse.org <pdt-dev-boun...@eclipse.org> de la part de thierry blind <thierrybl...@msn.com> Envoyé : vendredi 16 juin 2017 14:47 À : PDT Developers Objet : Re: [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen Ok, thank you for the explanations, Kaloyan! :) ________________________________ De : pdt-dev-boun...@eclipse.org <pdt-dev-boun...@eclipse.org> de la part de Kaloyan Raev <kaloya...@zend.com> Envoyé : vendredi 16 juin 2017 14:29 À : pdt-dev@eclipse.org Objet : Re: [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen Hi Thierry, > But before doing that, it would be really nice to have everybody's opinion in > next few days concerning the PDT 5.0 editor, just to be sure I'm not the only > one complaining about its state :) I agree that the PHP editor should be a top priority concerning the quality. > Can 2 users set same review flag and will it be shown that 2 users have set > the review flag? Every user can set its own review flag: "+" or "-". It will appear in Bugzilla as a separate flag with the user's ID next to it. > What is the best way to be informed about 5.0.1 patches? Add all team members > to the bug's CC list? You can request other users to review the bug by setting "?" for the review flag. In this case an additional text field appears and you can add the list of user emails. You can use the below list to request a review from all currently active committers, like I have just done this for bug 516350: zu...@w3des.net,thierrybl...@msn.com,iba...@qq.com,kaloya...@zend.com,micha...@zend.com,bartlomie...@zend.com<mailto:zu...@w3des.net,thierrybl...@msn.com,iba...@qq.com,kaloya...@zend.com,micha...@zend.com,bartlomie...@zend.com> Just copy these comma-separated list of emails in the text field. All requested user should receive a mail notification that they a requested to review the bug. Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> On 06/16/2017 05:06 PM, thierry blind wrote: Very good solution :) I'll create a bug report and a patch to revert the 2 culprit patches, and merge it later in the master branch. But before doing that, it would be really nice to have everybody's opinion in next few days concerning the PDT 5.0 editor, just to be sure I'm not the only one complaining about its state :) I never used these flags in bug reports. Are the flags "cumulative"? Can 2 users set same review flag and will it be shown that 2 users have set the review flag? What is the best way to be informed about 5.0.1 patches? Add all team members to the bug's CC list? Search bugs by "Target milestone"? It seems you can search (for example) by votes but not by flags. Thierry. ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> Envoyé : vendredi 16 juin 2017 13:08 À : pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> Objet : Re: [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen It's really good that we have this discussion on quality. I added 5.0.1 version in Bugzilla and PMI. Planned release date is June 28 - to coincide with the official Oxygen release. It won't be part of the PHP EPP, but everyone would be able to update to 5.0.1 at the same date. Unfortunately, the 5.0.1 update won't be automatically suggested by the PHP EPP, but users will need to manually add the respective update site. This is a point for improvement in the PHP EPP for the Oxygen.1 release. So, let's try the suggested "add code that is at least reviewed by 2 members" rule for this 5.0.1 release. Technically, all fixes should be first committed in master for the next feature release (today this is 5.1). Then we can cherry-pick the voted fixes into the maintenance branch (today this is stable-5.0). Voting for cherry-picking can happen in Bugzilla. We can use the review flag. Here is how I did it in the first candidate for 5.0.1: https://bugs.eclipse.org/bugs/show_bug.cgi?id=516350. A positive review should include actual testing. Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> On 06/16/2017 03:43 PM, thierry blind wrote: For the " add code that is at least reviewed by 2 members", it's of course for code after RC1 for example ;) I'll add another point about those 2 patches: they basically tried to fix long-standing problems in class ASTProvider. But some stacktraces before and after those patches look mostly the same, so main problems are still here. And it is even worse now! This class always had bugs, and it's not well-written (sorry to say that, complex code doesn't always mean good code), it's a real blackbox and a hell to debug. My conclusion is simple here: we should rewrite it to feat our needs, more simpler, with more debug traces, and ideally with understandable locking-system! As first step, no need to be optimal code, just "working code" ;) I would put that as top priority for 5.1, since Alex adds more and more (great) features to PDT that seem to depend on correct ASTProvider behavior ;) But that's just my opinion... ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de thierry blind <thierrybl...@msn.com><mailto:thierrybl...@msn.com> Envoyé : vendredi 16 juin 2017 12:25 À : PDT Developers Objet : Re: [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen Sorry Kaloyan, I mean patches https://git.eclipse.org/r/#/c/96253/ and https://git.eclipse.org/r/#/c/99102/ ! Yes it would be great to have some opinions from all team about using the 5.0 release, to see if we should revert those 2 patches. Opinions should only concern "writing PHP code using the PHP editor" ;) Personally I find it less good now, because you have validation "artifacts" and highlighting problems while coding. I don't say these problems are new, but they happen much more often now! I consider that as a regression. And totally agree for your last point, maybe we should even stop adding code earlier (before RC1), or add code that is at least reviewed by 2 members (other than the patch creator) for example. In the case of these 2 patches, time was too short to fully review them, even if first patch was created beginning may. Thierry. ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> Envoyé : vendredi 16 juin 2017 11:49 À : pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> Objet : Re: [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen Hi Thierry, As far as I can see https://git.eclipse.org/r/#/c/99312/ was just merged in master, i.e. it goes to 5.1-nightly, but not to 5.0. Gerrit Code Review - Eclipse Git repositories<https://git.eclipse.org/r/#/c/99312/> git.eclipse.org Loading Gerrit Code Review... ... Home; Privacy Policy; Terms of Use; Copyright Agent; Legal So, it is only https://git.eclipse.org/r/#/c/99102/ (the patch that we included in RC4) that actually worries you for PDT 5.0, right? Gerrit Code Review - Eclipse Git repositories<https://git.eclipse.org/r/#/c/99102/> git.eclipse.org Loading Gerrit Code Review... ... Home; Privacy Policy; Terms of Use; Copyright Agent; Legal BTW, you can download the final PHP EPP build from here: https://hudson.eclipse.org/packaging/job/oxygen.epp-tycho-build/318/artifact/org.eclipse.epp.packages/archive/ I suggest all of us do it and see if we are really happy with the result. If we still want to fix anything until the official release date (June 28) then I suggest that we target a 5.0.1 maintenance release at this date. I think it is also a good idea to be more careful during the RC phase of future releases. Perhaps, we can now build some good process of what goes into 5.0.1 and then reuse it for the RC phase of future releases. Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> Making it Easier to Write, Test and Deploy Complex Code | Rogue Wave <http://www.roguewave.com/> www.roguewave.com<http://www.roguewave.com> We’re the largest independent provider of cross-platform software development tools and embedded components in the world. Come see how we can help. On 06/16/2017 11:28 AM, thierry blind wrote: While speaking about bugs, would it be possible to revert somehow patches https://git.eclipse.org/r/#/c/99312/ and https://git.eclipse.org/r/#/c/99102/ ? I know I come (too) late for 5.0, but maybe for a 5.0.1 release? With more testing, they break things more than they solved in the PHP editor, for example: - compilation errors do not always disappear (annotations seem to be "stuck") - variable highlighting stop to work pretty fastly - unused variable highlighting (from the PDT extensions) also stop to work pretty fastly - and Alex finds more and more deadlocks (see https://git.eclipse.org/r/#/c/99343/) Only easy fix I found is to switch between editor tabs, it reset things afterwards (but problem will appear again). I think more and more we should have never merged these ones (we should have tested these patches much more, I was also too confident this time), but it's a bit too late to whine and complain about that ;) I'm worried about what the PDT users experience, we added so many great features, but the main component (the PHP editor) is now feeling more clumsy, I'm a bit sad about sad :( Because it can upset users and make them go away from PDT. Thierry. ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de Michal Niewrzal <michal.niewr...@roguewave.com><mailto:michal.niewr...@roguewave.com> Envoyé : vendredi 16 juin 2017 07:38 À : pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> Objet : [pdt-dev] Odp.: Fwd: PDT Eclipse Oxygen Try older PHP version, 5.6 or 7.0. I think there is bug with latest 7.1 Greetings, Michał Niewrzał | ZEND STUDIO Rogue Wave Software, Inc. Accelerating Great Code www.roguewave.com<http://www.roguewave.com/> / michal.niewr...@roguewave.com ________________________________ Od: pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> w imieniu użytkownika Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> Wysłane: 16 czerwca 2017 09:18:27 Do: pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> Temat: Re: [pdt-dev] Fwd: PDT Eclipse Oxygen I will rework, If you let me know how to make the Trace Failure appear :-) It just does not work for me in PDT. Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> On 06/15/2017 05:29 PM, Bartłomiej Łączkowski wrote: This sender failed our fraud detection checks and may not be who they appear to be. Learn about spoofing<http://aka.ms/LearnAboutSpoofing> Feedback<http://aka.ms/SafetyTipsFeedback> Cool :) I think that it would be nice to have one failing test in PHPUnit view selected to see some failure trace in "Failure Trace" pane. On 6/15/2017 4:13 PM, Kaloyan Raev wrote: How about this? Screenshot: attached Headline: Test Code and Manage Dependencies with PHP Development Tools Details: The PHP support in Eclipse improved by adding integration with PHPUnit for testing PHP code, and with Composer for managing PHP dependencies. Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> On 06/15/2017 02:00 PM, Dawid Pakuła wrote: I think both, composer editor plus phpunit after test run ;) Both with project explorer. -- Dawid Pakuła On 15 June 2017 at 10:27:27, Kaloyan Raev (kaloya...@zend.com<mailto:kaloya...@zend.com>) wrote: Thanks, Thierry! Any other opinions? If we should pick between Composer and PHPUnit, what would be your choice? Kaloyan Raev | Zend Studio Team Lead Rogue Wave Software, Inc. Innovate with Confidence M +359 887 648 663 www.roguewave.com<http://www.roguewave.com> / kaloyan.r...@roguewave.com<mailto:kaloyan.r...@roguewave.com> On 06/14/2017 12:14 PM, thierry blind wrote: Even so, what feature should we highlight? What would a PHP developer like the most in PDT 5.0? :) What is exclusive to PDT you don't find in other IDE yet? Globally debugging&testing is more powerful in PDT yet. PHP profiling is also a great new feature, but only for Zend debugger (and I'm using Xdebug "sadly"). Supporting composer projects is also a big step. Hard to choose, maybe we should do some vote here ;) ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> Envoyé : mercredi 14 juin 2017 08:53 À : pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> Objet : Re: [pdt-dev] Fwd: PDT Eclipse Oxygen True, it's impossible to have all new features and improvements in a single screenshot. I think the idea is to highlight one new thing that would encourage people to try PDT 5.0 (or Eclipse Oxygen in general). We have an article, a webinar and a N&N page to cover all the rest. For example, if you look at https://www.eclipse.org/oxygen/, JDT are promoting an improvement in the debugger. [https://www.eclipse.org/eclipse.org-common/themes/solstice/public/images/logo/eclipse-200x200.png]<https://www.eclipse.org/oxygen/> Eclipse Oxygen<https://www.eclipse.org/oxygen/> www.eclipse.org<http://www.eclipse.org> Eclipse is probably best known as a Java IDE, but it is more: it is an IDE framework, a tools framework, an open source project, a community, an eco-system, and a foundation. On 06/14/2017 11:44 AM, thierry blind wrote: Hi Kaloyan, that's not an easy task :) if you can do a screenshot for a php composer projet with an opened php 7.1 file having PHP syntax check and validation errors, with a PHPUnit tab, a Built-in PHP Server tab, or some PHP profiling tabs, you'll show some nice new features :) Thierry. ________________________________ De : pdt-dev-boun...@eclipse.org<mailto:pdt-dev-boun...@eclipse.org> <pdt-dev-boun...@eclipse.org><mailto:pdt-dev-boun...@eclipse.org> de la part de Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> Envoyé : mercredi 14 juin 2017 07:32 À : PDT Developers Objet : [pdt-dev] Fwd: PDT Eclipse Oxygen Hi Team, We have a request by the Eclipse Marketing to provide one screenshot of PDT 5.0 for the Oxygen page. We have many new features and improvements in PDT 5.0. How would the single screenshot look like? Any ideas? Thanks, Kaloyan -------- Forwarded Message -------- Subject: PDT Eclipse Oxygen Date: Tue, 13 Jun 2017 11:16:47 -0400 From: katharine hirsch <kat.hir...@eclipse-foundation.org><mailto:kat.hir...@eclipse-foundation.org> To: Kaloyan Raev <kaloya...@zend.com><mailto:kaloya...@zend.com> CC: Ian Skerrett <ian.skerr...@eclipse-foundation.org><mailto:ian.skerr...@eclipse-foundation.org> Hi Kaloyan, I hope all is well with you! We are in the midst of preparing the eclipse.org/oxygen<http://eclipse.org/oxygen> web page for June 28, and we were wondering if you'd be able to assist with a cool screenshot of PDT 5.0 in Oxygen that we could add to the screenshot section on the web page! We'd need a crisp high resolution screen shot, with a short description of what you're showing us, and a header. Is this something you could provide in the next day or so? Take a look at the site for examples, and let me know if you have questions. Thank you! Kat -- Kat Hirsch Marketing and Event Specialist Eclipse Foundation E: kat.hir...@eclipse-foundation.org<mailto:kat.hir...@eclipse-foundation.org> T: (613) 224-9461 x 242 _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev [https://ipmcdn.avast.com/images/icons/icon-envelope-tick-round-orange-animated-no-repeat-v1.gif]<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=icon> Virus-free. www.avast.com<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=link> _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev _______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org<mailto:pdt-dev@eclipse.org> To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev
_______________________________________________ pdt-dev mailing list pdt-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/pdt-dev
