oh YES, you are right, I swapped those unintentionally! <div><br></div><div>probably explains your bug with chapters? I used</div><div><br></div><div>$ git diff 66e7bfbe cinelerra/bdwrite.C</div><div><br></div><div>and yes, please switch cases 2/3 and 4 as they originally were... so it will look like this:</div><div><br></div><div><br></div><div>@@ -1905,24 +1916,24 @@ write()</div><div><br></div><div>   bs.write(stream_type, 8);</div><div>   switch (stream_type) {</div><div>-  case 0x01:</div><div>+  case 1:</div><div>     bs.write(pid, 16);</div><div>     break;</div><div><br></div><div>-  case 0x02:</div><div>-  case 0x04:</div><div>+  case 2:</div><div>+  case 4:</div><div>     bs.write(subpath_id, 8);</div><div>     bs.write(subclip_id, 8);</div><div>     bs.write(pid, 16);</div><div>     break;</div><div><br></div><div>-  case 0x03:</div><div>+  case 3:</div><div>     bs.write(subpath_id, 8);</div><div>     bs.write(pid, 16);</div><div>     break;</div><div><br></div><div>   default:</div><div>-    fprintf(stderr, "unrecognized stream type %02x\n", stream_type);</div><div>+    fprintf(stderr, "unrecognized mpls stream type %02x\n", stream_type);</div><div>     break;</div><div>   };</div><div>   bs.padb(9 - strm.bs_posb(bs));</div><div>@@ -1935,6 +1946,7 @@ write()</div><div>   case BLURAY_STREAM_TYPE_VIDEO_MPEG2:</div><div>   case BLURAY_STREAM_TYPE_VIDEO_VC1:</div><div><br></div><div><br></div><div>=== </div><div><br></div><div>i can make real fix patch if you wish<br><br>On Monday, April 18, 2022, Phyllis Smith <<a href="mailto:phylsmith2017@gmail.com">phylsmith2017@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">Andrew,</div><div class="gmail_default" style="font-size:small">Tomorrow I am going to make one last test (still without chapters) and check this mini_pile into GIT, but 1 more question.</div><div class="gmail_default" style="font-size:small">From megapile_10, 0015-Attempt-at-fixing-<wbr>bdwrite-stream_type-coding_<wbr>type-co.patch, I think the case renumbering of 3 and 4 may be incorrect.</div><div class="gmail_default" style="font-size:small">Currently the code for case 3 and 4 is different than what it used to be  -- I think these numbers mean something (although 0x03 versus 3 is OK and 0x04 versus 4 is OK).  See this diff in bdwrite.C :</div><div class="gmail_default" style="font-size:small"><span style="font-family:monospace"><b>Before =  </b>                           <wbr>             <b>After =</b><br></span></div><div class="gmail_default" style="font-size:small"><span style="font-family:monospace">  case 0x04:                         <wbr>              case 4:<br>    bs.write(subpath_id, 8);                        bs.write(subpath_id, 8);<br>    bs.write(subclip_id, 8);                        bs.write(pid, 16);  <br>    bs.write(pid, 16);                          <wbr>    break;<br>    break;<br><br>  case 0x03:                         <wbr>               case 3:<br>    bs.write(subpath_id, 8);                          bs.write(subpath_id, 8);<br>    bs.write(pid, 16);                          <wbr>      bs.write(subclip_id, 8);<br>    break;                        <wbr>                    bs.write(pid, 16);</span></div><div class="gmail_default" style="font-size:small"><span style="font-family:monospace">                              <wbr>                        break:</span><br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Could you check this??<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 17, 2022 at 12:01 PM Andrew Randrianasulu <<a href="mailto:randrianasulu@gmail.com" target="_blank">randrianasulu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br><br>On Sunday, April 17, 2022, Phyllis Smith <<a href="mailto:phylsmith2017@gmail.com" target="_blank">phylsmith2017@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div style="font-size:small"><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:small">The
 2 modifications I made are in bdcreate.C to temporarily comment out the
 choices of lpcm and truehd as neither 1 works for me (see error 
messages in previous email).</div></div></blockquote><div><br></div><div>those mesages come from players themselves? <br></div></blockquote><div><br></div></div>No, from bdwrite -- see the last line in the terminal output below for the error message when I choose "truehd" and then "lpcm".  I verified that 0004-Improve-truehd-decoder-<wbr>encoder-from-ffmpeg.git.patch is in as well as all of the patches as stated earlier.  <br></div></div></div></blockquote><div><br></div><div><br></div><div>be sure there is no old bdwrite binary in PATH? <br></div></blockquote><div><span class="gmail_default" style="font-size:small">It is rare that I change $PATH from or even add cnelerra:   </span> /root/.local/bin:/root/bin:/<wbr>usr/local/sbin:/usr/local/bin:<wbr>/usr/sbin:/usr/bin</div></div></div></blockquote><div><br></div><div><br></div><div>still, old bdwrite may lay here.. I say with some confidence because fully-patched bdwrite binary even output different error message for stream type! </div><div><br></div><div>try to add some your own string ( fprintf ("my cool string\n";) into code of bdwrite (right into main {} function) and see  if it printed when script executes? </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div></div><div><br></div><div><br></div><div> PATH=/root/.local/bin:/root/<wbr>bin:/usr/local/sbin:/usr/<wbr>local/bin:/usr/sbin:/usr/bin:/<wbr>tmp/cinelerra-5.1/bin</div></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-family:monospace"></span><span style="font-family:monospace"></span><span style="font-family:monospace"></span><span style="font-family:monospace"></span><span style="font-family:monospace"></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div style="font-size:small">However, the choice of "tsmuxer" works, or at least does something and creates no problem but I do not see or hear any difference if checked off or on.<br></div></div></blockquote>you need to have tsmuxer binary in $PATH   to see it used <br></blockquote><div><span class="gmail_default" style="font-size:small">I wonder why the output is different when I check the tsmuxer box than when I don't?</span></div><br></div></div></blockquote><div>Because I do not check for tsmuxer existence in C code, and just  write universal sh script doing check for us? <br></div></blockquote><div><span class="gmail_default" style="font-size:small">Never mind -- it is different just when running it twice so there must be timestamps in it.</span></div><div><span class="gmail_default" style="font-size:small">I did download tsmuxer source code, but when it said I needed "ninja", I stopped and just laughed.  Maybe later I will try again.</span> </div></div></div></blockquote><div><br></div><div>yeah, I have this installed for mesa3d builds mostly... </div>
</blockquote></div>
</blockquote></div>