<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-06-08 13:06 GMT+02:00 Sergey Isakov <span dir="ltr"><<a href="mailto:isakov-sl@bk.ru" target="_blank">isakov-sl@bk.ru</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Hi Christian,</div><div>I am fully agree with you that I have to mention all contributors for this work.</div><div>There is just a little problem for me is how to divide and rebase my sources to be applied to... to what?</div><div>Currently I modified wine-stage sources and then make diff related to wine-hq. Then I extract only hunks related to the topic.</div><div>I checked it can be applied.</div><div>Wine-stage exists as patches not as complete sources and I can't do diff related to it.</div></div></blockquote><div><br></div><div> To manage patchset I suggest to use stacked-git or similar tools and import the needed patches from wine-staging.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Also I don't know what is mean "one patch for change". This is one change. I know what is one idea so why I made two patches for these two ideas.</div></div></blockquote><div><br></div><div>Use a smaller granularity for your patches as I did. "one patch per change" means "don't use one patch for unrelated changes".</div><div>Fixing alignment in D3DXCheckTextureRequirements is not the same as "<span style="white-space:pre-wrap;color:rgb(0,0,0)">Fix several issues in save_dds_surface_to_memory" which is not the same as the other patches.</span></div><div><span style="white-space:pre-wrap;color:rgb(0,0,0)">You will be asked to do that anyway as this is needed to ease review and have patches applied.</span></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>About libtxc_dxtn I already did the job to embed them into wined3d by sources. If you want I can show how.</div><div>So</div><span class=""><div><blockquote type="cite"><div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">- wined3d: Add support for DXTn software decoding through libtxc_dxtn. [rev 3]</span></font></pre></div></blockquote></div></span><div>--</div><div>The library is not linked. It compiled inside.</div><span class=""><div><blockquote type="cite"><div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">- wined3d: add DXT1 to B4G4R4A4, DXT1 to B5G5R5A1 and DXT3 to B4G4R4A4 conversion.</span></font></pre></div></blockquote></div></span><div>--</div><div>Only conversion to and from A8B8G8R8 used.</div><span class=""><div><blockquote type="cite"><div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">- wined3d: Improve DXTn support</span></font></pre></div></blockquote></div></span><div>--</div><div>On the way. There is no in my commit.</div><span class=""><div><blockquote type="cite"><div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">and export conversion functions for d3dx9_36.</span></font></pre></div></blockquote></div></span><div>--</div><div>Currently I commit this as a stub. (return NULL;)</div><div><br></div><div>About patch1 I shown many lacks in existing procedures. I corrected them, add missing and tested. </div><div>OK, let divide on several pieces what is what, who made this, and how to join or split them.</div><div><br></div><div>If anybody help me to do the work I will be very thankful.</div></div></blockquote><div><br></div><div>I'm from wine-staging so it's better to see with winehq guys.</div><div>You may consider integrating your changes in wine-staging first if you like and them into pure wine then.</div><div>This way you can improve you patches and don't bother with the wine-staging patches you base your work on so your work have better chance to be accepted in pure wine afterwards.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>Anyway. Is there any movements to convince wine developer to accept DXTn support into mainstream? Is there anybody consults with justice?</div></div></blockquote><div><br></div><div>I don't know their position about that. There were some use of libtxc_dxtn in the past but removed afterwards. This may be for legal reason or not. The best thing is to ask them.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>AFAIK there is no problem for wine to support DXTn. It is a problem for Game developer to create such textures while wine is allowed to interpret any input.</div></div></blockquote><div><br></div><div>Not sure I understand but yeah DXTn is needed.</div><div>  </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>Also it will be OK for me if this patchset will be accepted into stage because it depends on it. I just can't understand how to make diff in this case.</div></div></blockquote><div><br></div><div>You mean wine-staging? If so see above.</div><div><br></div><div>Christian</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>Yours,</div><div>Sergey</div><div><div class="h5"><br><div><div>On 08.06.2015, at 12:48, Christian Costa wrote:</div><br><blockquote type="cite"><div dir="ltr"><pre style="word-wrap:break-word"><font color="#000000"><span style="white-space:pre-wrap">Hi Sergey,

He are my wine-staging patches you included in your patches:

Patch 1:
- d3dx9_36: Align texture dimensions to block size for compressed textures in D3DXCheckTextureRequirements.
- d3dx9_36: Add support for FOURCC surface to save_dds_surface_to_memory
- d3dx9_36: Fix several issues in save_dds_surface_to_memory.
- d3dx9_36: Improve D3DXSaveTextureToFile to save simple texture to dds file.

Patch 2:
- d3dx9_36: Add dxtn support.

It seems also that patch 2 uses some code from Michael Muller's patches and it seems you intend to use them for the dxtn conversion in wined3d. 
- wined3d: Add support for DXTn software decoding through libtxc_dxtn. [rev 3]
- wined3d: add DXT1 to B4G4R4A4, DXT1 to B5G5R5A1 and DXT3 to B4G4R4A4 conversion.
- wined3d: Improve DXTn support and export conversion functions for d3dx9_36.

When using code from others, you need to mention its origin and and add the proper references to the authors.
Moreover, you squashed different changes into single patches while the rule is to have one patch per change. I would like also the granularity of my patches to be kept.
This is much better to discuss how to upstream these patches with the authors at #wine-staging.
Note that before doing some DXTN work you should first implement the low level functions based on libtxc_dxtn and make sure they can be accepted (wined3d exports, legal issue, ...).

Christian</span></font></pre><div class="gmail_extra"><br><div class="gmail_quote">2015-06-08 9:09 GMT+02:00 Nikolay Sivov <span dir="ltr"><<a href="mailto:bunglehead@gmail.com" target="_blank">bunglehead@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On 08.06.2015 10:01, Sergey Isakov wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
It is hard to distill what part of codes comes from where. My patches depends on previous patches and so they must be included too.<br>
OK, all copylefts must be included if I will know how to do this.<br>
If Christian Costa will say what to do I will do this.<br>
</blockquote>
<br>
Sure, let's hear his opinion.<div><div><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
On 08.06.2015, at 9:43, Nikolay Sivov wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On 07.06.2015 23:02, Sergey Isakov wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I decided to improve builtin d3dx9_36 library with support of more file<br>
formats, texture formats, volume textures.<br>
</blockquote>
<br>
It's not your work though, it looks to me like 3 or so patches from Christian Costa squashed together. Why didn't you mention it?<br>
</blockquote>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>
</blockquote></div><br></div></div></div></blockquote></div><br></div></div>