<div dir="ltr"><div>I think your patch also needs a rustdoc run-time dependency on libjs-jquery.<br>If we're going to remove it from the repacked upstream source, then we can also remove the un-minified source in debian/missing-sources/</div><div><br><span style="line-height:1.5">For comparison (slightly different situation), in cargo packaging we just replaced the symlink after building the docs:</span><br></div><div>  <a href="https://anonscm.debian.org/cgit/pkg-rust/cargo.git/tree/debian/rules#n62">https://anonscm.debian.org/cgit/pkg-rust/cargo.git/tree/debian/rules#n62</a></div><div><br></div><div>---</div><div><br></div><div>Hrm.  I feel like this is a user-visible change, and therefore something we might not want to do without a good reason.  On the other hand, anyone who publishes HTML docs to another host *should* be resolving out-of-sub-tree symlinks anyway.  So I'm still on the fence :/</div><div><br></div><div>A possible alternative might be to read the source file from /usr/share/javascript at generation time (rather than embed it), but still generate a regular jquery.js output file (ie: copy, not symlink).  Less space-efficient in output, but makes it look closer to what upstream generates.</div><div><br></div><div>I definitely agree that we should at least build-depend on libjs-jquery and replace the embedded copy with the libjs-jquery version.  I have a very vague memory that someone tried this and it wasn't as easy as just dropping in a (possibly different) version of jquery however - which invalidates all of the above alternatives.  Anyone remember the details?</div><div><br></div><div> - Gus</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, 6 Aug 2016 at 05:15 Ximin Luo <<a href="mailto:infinity0@debian.org">infinity0@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ximin Luo:<br>
> <a href="https://anonscm.debian.org/cgit/pkg-rust/rust.git/commit/?h=_experiment_jquery&id=6b3175c6" rel="noreferrer" target="_blank">https://anonscm.debian.org/cgit/pkg-rust/rust.git/commit/?h=_experiment_jquery&id=6b3175c6</a><br>
<br>
Correction, I updated it a bit. URL to the latest version (dynamic):<br>
<br>
<a href="https://anonscm.debian.org/cgit/pkg-rust/rust.git/commit/?h=_experiment_jquery&id=_experiment_jquery" rel="noreferrer" target="_blank">https://anonscm.debian.org/cgit/pkg-rust/rust.git/commit/?h=_experiment_jquery&id=_experiment_jquery</a><br>
<br>
X<br>
<br>
--<br>
GPG: ed25519/56034877E1F87C35<br>
GPG: rsa4096/1318EFAC5FBBDBCE<br>
<a href="https://github.com/infinity0/pubkeys.git" rel="noreferrer" target="_blank">https://github.com/infinity0/pubkeys.git</a><br>
<br>
_______________________________________________<br>
Pkg-rust-maintainers mailing list<br>
<a href="mailto:Pkg-rust-maintainers@lists.alioth.debian.org" target="_blank">Pkg-rust-maintainers@lists.alioth.debian.org</a><br>
<a href="https://lists.alioth.debian.org/mailman/listinfo/pkg-rust-maintainers" rel="noreferrer" target="_blank">https://lists.alioth.debian.org/mailman/listinfo/pkg-rust-maintainers</a><br>
</blockquote></div>