Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
On 03/12/13 18:58, Iván Briano wrote: > On Tue, Dec 3, 2013 at 3:10 PM, Stefan Schmidt > wrote: >> Hello. >> >> On Mon, 2013-12-02 at 11:02, Stefan Schmidt wrote: >>> >>> Please be aware that we don't want to update NEWS and Changelog with >>> every commit in this cycle anymore. I have just seen some commits >>> doing that. They only copy the commit subject into two more files. >>> >>> I will go through the commit log before cutting tarballs and update >>> them accordingly. So please put your energy into writing good commit >>> messages instead of updating two moe files with the same content. >> >> Seems people are still struggling what I expect from them in the >> commit message to make my life easier. >> >> I don't have a template for you as I don't want to cut down your >> creativity. :) What I can give you is a few questions you can ask >> yourself when writing it. >> >> o Is it a bug fix? If yes please mention this somehow in the commit >> message e.g. evas: Fix length check to avoid slow path in evas line >> That would perfectly indicate to me that this is a bug fix. >> >> o Does the bug have a phab ticket? (It should) Please mention the phab >> ticket number. Same for the coverity ID. >> >> o Bug fixes for features that just merged in this cycle are good but >> long standing bugs from older releases are more important to get >> mentioned. >> >> o For bigger features which might span over several patches I really >> would appreciate a good high level description that goes either >> directly into the release notes wiki page or in one of the first >> commits. If you do the later please point this out to me so I can >> bring it over to the release note page. (Hint: Directly put it into >> the wiki makes both our life easier.) >> >> Thats is for now. Once I find more problems I will let you know. For >> now I would like to keep this lean and grow more naturally. >> > > If I may add, commit messages that describe obvious code don't help, > but adding the reason why the change matters does. So don't just describe > what you did, but tell us why, what was the problem, why did that part > of the code > caused it, why does your change fix it. > > This helps also not only to review the actual patch but to find out if the fix > is the best we can come up with, or if maybe it's not the right fix > but it's covering > a deeper problem. Yes, I hate those commit messages! "Added an if to check some-crazy-shit is equal to 8 in some-other-crazy-shit." -- Tom. -- Sponsored by Intel(R) XDK Develop, test and display web and hybrid apps with a single code base. Download it for free now! http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
On Tue, 3 Dec 2013 18:10:51 +0100 Stefan Schmidt wrote: > Hello. > > On Mon, 2013-12-02 at 11:02, Stefan Schmidt wrote: > > > > Please be aware that we don't want to update NEWS and Changelog with > > every commit in this cycle anymore. I have just seen some commits > > doing that. They only copy the commit subject into two more files. > > > > I will go through the commit log before cutting tarballs and update > > them accordingly. So please put your energy into writing good commit > > messages instead of updating two moe files with the same content. > > Seems people are still struggling what I expect from them in the > commit message to make my life easier. > > I don't have a template for you as I don't want to cut down your > creativity. :) What I can give you is a few questions you can ask > yourself when writing it. > > o Is it a bug fix? If yes please mention this somehow in the commit > message e.g. evas: Fix length check to avoid slow path in evas line > That would perfectly indicate to me that this is a bug fix. > > o Does the bug have a phab ticket? (It should) Please mention the phab > ticket number. Same for the coverity ID. > > o Bug fixes for features that just merged in this cycle are good but > long standing bugs from older releases are more important to get > mentioned. > > o For bigger features which might span over several patches I really > would appreciate a good high level description that goes either > directly into the release notes wiki page or in one of the first > commits. If you do the later please point this out to me so I can > bring it over to the release note page. (Hint: Directly put it into > the wiki makes both our life easier.) > > Thats is for now. Once I find more problems I will let you know. For > now I would like to keep this lean and grow more naturally. You probably should merge this into https://phab.enlightenment.org/w/efl_and_elementary_1_9/ -- A big old stinking pile of genius that no one wants coz there are too many silver coated monkeys in the world. signature.asc Description: PGP signature -- Sponsored by Intel(R) XDK Develop, test and display web and hybrid apps with a single code base. Download it for free now! http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
On Tue, Dec 3, 2013 at 3:10 PM, Stefan Schmidt wrote: > Hello. > > On Mon, 2013-12-02 at 11:02, Stefan Schmidt wrote: >> >> Please be aware that we don't want to update NEWS and Changelog with >> every commit in this cycle anymore. I have just seen some commits >> doing that. They only copy the commit subject into two more files. >> >> I will go through the commit log before cutting tarballs and update >> them accordingly. So please put your energy into writing good commit >> messages instead of updating two moe files with the same content. > > Seems people are still struggling what I expect from them in the > commit message to make my life easier. > > I don't have a template for you as I don't want to cut down your > creativity. :) What I can give you is a few questions you can ask > yourself when writing it. > > o Is it a bug fix? If yes please mention this somehow in the commit > message e.g. evas: Fix length check to avoid slow path in evas line > That would perfectly indicate to me that this is a bug fix. > > o Does the bug have a phab ticket? (It should) Please mention the phab > ticket number. Same for the coverity ID. > > o Bug fixes for features that just merged in this cycle are good but > long standing bugs from older releases are more important to get > mentioned. > > o For bigger features which might span over several patches I really > would appreciate a good high level description that goes either > directly into the release notes wiki page or in one of the first > commits. If you do the later please point this out to me so I can > bring it over to the release note page. (Hint: Directly put it into > the wiki makes both our life easier.) > > Thats is for now. Once I find more problems I will let you know. For > now I would like to keep this lean and grow more naturally. > If I may add, commit messages that describe obvious code don't help, but adding the reason why the change matters does. So don't just describe what you did, but tell us why, what was the problem, why did that part of the code caused it, why does your change fix it. This helps also not only to review the actual patch but to find out if the fix is the best we can come up with, or if maybe it's not the right fix but it's covering a deeper problem. > regards > Stefan Schmidt > > -- > Rapidly troubleshoot problems before they affect your business. Most IT > organizations don't have a clear picture of how application performance > affects their revenue. With AppDynamics, you get 100% visibility into your > Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! > http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk > ___ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
Hello. On Mon, 2013-12-02 at 11:02, Stefan Schmidt wrote: > > Please be aware that we don't want to update NEWS and Changelog with > every commit in this cycle anymore. I have just seen some commits > doing that. They only copy the commit subject into two more files. > > I will go through the commit log before cutting tarballs and update > them accordingly. So please put your energy into writing good commit > messages instead of updating two moe files with the same content. Seems people are still struggling what I expect from them in the commit message to make my life easier. I don't have a template for you as I don't want to cut down your creativity. :) What I can give you is a few questions you can ask yourself when writing it. o Is it a bug fix? If yes please mention this somehow in the commit message e.g. evas: Fix length check to avoid slow path in evas line That would perfectly indicate to me that this is a bug fix. o Does the bug have a phab ticket? (It should) Please mention the phab ticket number. Same for the coverity ID. o Bug fixes for features that just merged in this cycle are good but long standing bugs from older releases are more important to get mentioned. o For bigger features which might span over several patches I really would appreciate a good high level description that goes either directly into the release notes wiki page or in one of the first commits. If you do the later please point this out to me so I can bring it over to the release note page. (Hint: Directly put it into the wiki makes both our life easier.) Thats is for now. Once I find more problems I will let you know. For now I would like to keep this lean and grow more naturally. regards Stefan Schmidt -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
Hello. On Mon, 2013-12-02 at 18:24, Rafael Antognolli wrote: > On Mon, Dec 2, 2013 at 8:02 AM, Stefan Schmidt > wrote: > > Hello. > > > > Please be aware that we don't want to update NEWS and Changelog with > > every commit in this cycle anymore. I have just seen some commits > > doing that. They only copy the commit subject into two more files. > > > > I will go through the commit log before cutting tarballs and update > > them accordingly. So please put your energy into writing good commit > > messages instead of updating two moe files with the same content. > > > > For bigger features, whcih may even span over several patches, it > > would be great if you could write a short summary either in one of the > > patches or on the 1.9 release notes wiki page: > > What about the 1.8 branch and micro releases? Nothing new in this cycle about it. We will provide stable releases over time. Most likely more often in the beginning and less in the end of the cycle. Raster already crate dthe 1.8 branches and did efl 1.8.1 I will hopefully follow up with some more updates on Friday. regards Stefan Schmidt -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
On Mon, Dec 2, 2013 at 8:02 AM, Stefan Schmidt wrote: > Hello. > > Please be aware that we don't want to update NEWS and Changelog with > every commit in this cycle anymore. I have just seen some commits > doing that. They only copy the commit subject into two more files. > > I will go through the commit log before cutting tarballs and update > them accordingly. So please put your energy into writing good commit > messages instead of updating two moe files with the same content. > > For bigger features, whcih may even span over several patches, it > would be great if you could write a short summary either in one of the > patches or on the 1.9 release notes wiki page: What about the 1.8 branch and micro releases? > https://phab.enlightenment.org/w/efl_and_elementary_1_9_release_announcement/ > > Please also refer to > https://phab.enlightenment.org/w/efl_and_elementary_1_9/#release-news-and-changes > > regards > Stefan Schmidt aka the man with the slapping trout for the 1.9 cycle > > -- > Rapidly troubleshoot problems before they affect your business. Most IT > organizations don't have a clear picture of how application performance > affects their revenue. With AppDynamics, you get 100% visibility into your > Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! > http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk > ___ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel -- Rafael Antognolli -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
Re: [E-devel] NEWS and ChangeLog updates in 1.9 cycle
On Mon, Dec 2, 2013 at 7:02 PM, Stefan Schmidt wrote: > Hello. > > Please be aware that we don't want to update NEWS and Changelog with > every commit in this cycle anymore. I have just seen some commits > doing that. They only copy the commit subject into two more files. > Great! It would be painful for you but it really helps all the developers. Thanks. Daniel Juyung Seo (SeoZ) > > I will go through the commit log before cutting tarballs and update > them accordingly. So please put your energy into writing good commit > messages instead of updating two moe files with the same content. > > For bigger features, whcih may even span over several patches, it > would be great if you could write a short summary either in one of the > patches or on the 1.9 release notes wiki page: > > > https://phab.enlightenment.org/w/efl_and_elementary_1_9_release_announcement/ > > Please also refer to > > https://phab.enlightenment.org/w/efl_and_elementary_1_9/#release-news-and-changes > > regards > Stefan Schmidt aka the man with the slapping trout for the 1.9 cycle > > > -- > Rapidly troubleshoot problems before they affect your business. Most IT > organizations don't have a clear picture of how application performance > affects their revenue. With AppDynamics, you get 100% visibility into your > Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics > Pro! > http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk > ___ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
[E-devel] NEWS and ChangeLog updates in 1.9 cycle
Hello. Please be aware that we don't want to update NEWS and Changelog with every commit in this cycle anymore. I have just seen some commits doing that. They only copy the commit subject into two more files. I will go through the commit log before cutting tarballs and update them accordingly. So please put your energy into writing good commit messages instead of updating two moe files with the same content. For bigger features, whcih may even span over several patches, it would be great if you could write a short summary either in one of the patches or on the 1.9 release notes wiki page: https://phab.enlightenment.org/w/efl_and_elementary_1_9_release_announcement/ Please also refer to https://phab.enlightenment.org/w/efl_and_elementary_1_9/#release-news-and-changes regards Stefan Schmidt aka the man with the slapping trout for the 1.9 cycle -- Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk ___ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
