-
nsITobin
good morning WG9s
-
WG9s
good morning!
-
nsITobin
all I need is to know what you want me to make this Buoy do
postimg.cc/crdZQsDb
-
WG9s
nsITobin: if you are asking me please send this in other than an image. It is completely illegible to me
-
nsITobin
can you click on it to zoom it out?
-
nsITobin
or is postimage just crap
-
nsITobin
here is imgbb
ibb.co/JxXSCGT
-
nsITobin
its just a window with a menubar a browser control and devtools
-
WG9s
i trid soming before i asked for other than image
-
WG9s
the subwindow with the actual error is illegiable and zomming is of zero help
-
WG9s
cantl you just give me the test of the error or a complete log from the build or something
-
nsITobin
I am confused.
-
nsITobin
I need to reboot back to windows
-
frg_Away
hi nsITobin
-
nsITobin
sup
-
nsITobin
just poking around about pages on this Buoy component i whipped up a little while back on central.. considering hooking up that overlay emulation and spawning navigator.xhtml
-
nsITobin
last I attempted it was around 122
-
njsg
0730|12:44:29 <+nsITobin> here is imgbb
ibb.co/JxXSCGT <-
i.ibb.co/CJxSWYy/image.png if it helps
-
nsITobin
its cool thanks njsg
-
njsg
but I don't think this was about an error, just showing it working with comm-central?
-
nsITobin
ye .. i need to work on being more clear
-
njsg
postimg.cc has a download image link that, at least with "open in browser" can be opened as an image
-
njsg
ibb.co has an image_src <link> that can be reached from the website navigation bar
-
njsg
(under "More")
-
nsITobin
postimg seems to go down a lot or has lately
-
nsITobin
I just need a paste/img bin
-
nsITobin
-
WG9s
nsITobin: are you applying the gitlab seamonkey-central-patches patch queues?
-
nsITobin
yeah
-
nsITobin
that does a lot
-
WG9s
OK just on first blush my take was maybe not appling the patches for bug TOP-1611647-1-rename-xul-suite.patch and TOP-1611647-2-fix-xul-references-suite.patch
-
WG9s
shich have to do with changing comm-central to handle change .xul files to .xhtml
-
nsITobin
I think I can make suiteglue partly work by jsm'ing it but that is quick on the slate for removal and sys.mjs would ahve to swiftly follow to get seamonkey to start normally
-
nsITobin
least enough to spew errors
-
nsITobin
some of these issues are gonna still bare down with backports
-
WG9s
but since errors seemded to be about .xhtml files i suspected this
bug 2 parts one rename all files with .xul extensions to .xhtml the second hanges the filenmes in all the references
-
WG9s
if you are adding more .xul files you might need my script that redoes the renames and rereferences .xul files do not work in central
-
WG9s
the sehll script here takes new changes into account to fix the patch for this
gitlab.com/frg/seamonkey-central-pa…scripts/xul2xhtml.sh?ref_type=heads
-
nsITobin
WG9s: I been studying very closely what they been up to
dpaste.org/oGGiG
-
nsITobin
it irks me
-
nsITobin
but.. i learned
-
WG9s
anyway if doing central patches don;t use .xul extenyion use .xhtml instead
-
nsITobin
indeed
-
nsITobin
still seems dumb
-
nsITobin
an internal extension shouldn't REALLY matter
-
nsITobin
just what a few lines to make sure it opens in the right parser
-
WG9s
but .xhtml is a web standard thing .xul is a mozilla thing
-
nsITobin
yeah and about: protocol hides most chrome urls ;)
-
WG9s
so the stupid scritp changes all .sul files inthe repo to .xhtml and a scond patch that finds all referwnces in the code to .sul files and changes thenm to .xhtml
-
nsITobin
still this is about as XUL as xhtml can get anymore as near as I can figure from other examples throughout the codebase
-
nsITobin
they have some xbl replacements as xul customelements but that's just transitional
-
WG9s
you would think who cares but they removed the special code for .xul fiels form m-c becuase id you just rename them to .xhtml the code for that takes care of them
-
WG9s
so no longer need special case for .xul files
-
nsITobin
sad
-
WG9s
maybe they need to chage it to say there is only xytml
-
WG9s
shtml
-
WG9s
xhtml
-
WG9s
third time is the charm
-
nsITobin
register the domain name redirect it back to XUL
-
WG9s
;-)
-
nsITobin
WG9s: IF IT STOPPED HERE.. i MIGHT be willing to embrase it but we know it doesn't
-
nsITobin
that's the thing about modern mozilla
-
nsITobin
as a .. whatever it is
-
WG9s
I am big on redirects and meta refresh pages in the archive for my nightly builds becuase seems no way to do symlinks
-
nsITobin
symlinks can be a pain
-
nsITobin
i think it is funny out of all of it.. composer of all things is SLIGHTLY functional
-
WG9s
so like was about to send you an exapmle but seems the seamonkey project archive site is now down, or at least not reachable via version from the US
-
nsITobin
u'd think Mozilla would ressurect the editor ui for them selves for chrome html usage
-
WG9s
-
WG9s
nsITobin: so already a typo in my automation scipt change
-
WG9s
somehow pusddd instead of pushd
-
WG9s
well pushdd instead of pushd but you get the idea
-
nsITobin
frg_Away: I am at the mangling my own patches stage of my hg mq learning
-
frg_Away
have mq fun
-
nsITobin
i am :)
-
nsITobin
aaand a pull from c-c just busted me
-
nsITobin
says run mach tb-rust vendor do it and it fails
-
nsITobin
0:00.65 [INFO] Updating gkrust in our workspace
-
nsITobin
error: invalid escape sequence
-
nsITobin
expected `b`, `f`, `n`, `r`, `t`, `u`, `U`, `\`, `"`
-
nsITobin
--> Cargo.toml:23:23
-
nsITobin
|
-
nsITobin
23 | cmake = { path = "..\..\build\rust\cmake" }
-
nsITobin
GREAT system..
-
WG9s
seems frg broke something whn yu ceck-out c-c dot from changeset 30c2520e5b0ec3fe26c2c8c69f24352732f37463
-
WG9s
foit
-
WG9s
that is the last changeset from before when frg screwed the pooch
-
WG9s
not sure his stuff is the reason but looks suspicious
-
nsITobin
how does adding a few urls to debugqa do this?!
-
WG9s
not sure at all
-
nsITobin
looks like rust
-
WG9s
just bustage seems c-c only and only these 3 changes
-
nsITobin
now i get a toml.decoder.TomlDecodeError: Reserved escape sequence used (line 18 column 1 char 294)
-
WG9s
checking out the eatlier changeset did not help so must be a m-c change that broke something
-
WG9s
I suspect something was chacked in no m-c that broke enable thunderbird-rust
-
WG9s
try building with that disabled maybe?
-
WG9s
oh stuipd me tested earlier changes wrong
-
WG9s
testign correctly based on rgular chanset in about 5 mins
-
WG9s
but then about 20 mins later to know
-
nsITobin
oh well it IS just central anyway.. I don't need to get all I MUST DO THIS NAOW about it .. learned some more progressed fullilling one of the things I said i would do.. learned more about hg mq.. all in all a good day of hackin on codestuffs
-
WG9s
JUt seems to be boken when i ws trying to test tomething
-
WG9s
if it is the frg stuff i would think mostlikely
Bug 1909853 becuase it touches mailnews which is where the enable-thunderbird-rust comes into play
-
njsg
but was there a change in Cargo.toml?
-
nsITobin
yes
-
nsITobin
and a new rust component added
-
nsITobin
in cc
-
nsITobin
hold on i may be dumb and thowing WG9s off.. always a possibility
-
WG9s
but m builds are failing and so are the thunderbird builds failurews started with frg's 3 patches landed
-
WG9s
so unless something landed around same time on m-c that borke this is supect the feg patches
-
WG9s
frg
-
WG9s
nsITobin: frg is gone for the day i am trying to taise IanN_Away
-
nsITobin
well i was on rev 42116 i see another sync vendoreed rust libs with moziilla central, repackage thunderbird as a debian package, my ui patch, a chat patch and esr links to debugqa.. how can frg's patches break hg like is it something in the header? or
-
nsITobin
I am gonna try purging .mozillabuild .cargo and .rustup see if completely "fresh" it works
-
WG9s
well could caus the error i am seeing not sure why your weeoe looks differnet
-
njsg
wait, why the backslashes?
-
nsITobin
I am on windows
-
nsITobin
atm
-
nsITobin
but
-
nsITobin
aren't all paths in mozilla regular slashes
-
nsITobin
to the build system
-
nsITobin
save internal python
-
nsITobin
maybe that's why
-
WG9s
no if you are in windows seems either slash works sometimes
-
WG9s
this is one of the things i complained about in the dos days
-
nsITobin
Rust dependencies are out of sync. Run `mach tb-rust vendor`
-
WG9s
Microsoft decided to go the DWC VMS ay of things fielnames are case insensitive options use a / as the dleimiter so not avilable to use as a path delimeter
-
njsg
DOS itself could handle another separator, third party software is probablya trickier quesiton
-
nsITobin
run it
-
nsITobin
it fails
-
njsg
now Windows NT, my guess would be it translates internally too?
-
WG9s
dos version one allowed you to change the command option delimeter and i ran at the time changing it to - but in vrsion 2 they elimiated that and came up with path delimiter is \
-
WG9s
and i don;t remeber there being a way to change
-
WG9s
they seems to be tageting making it be a DEC VMS replacement on a smaller platform
-
WG9s
all worked berrer in the msdos they stole to sell to IBM
-
WG9s
tomake pcdos
-
njsg
nsITobin: is there more than one error about escape sequences? because Cargo.toml:23:23 looks a bit off
-
njsg
I mean, the code printed afterwards has a *valid* escape at column 23
-
nsITobin
well my build output is a bit manged because i am not using the msys2 terminal but just through oldstyle conhost..
-
nsITobin
-
nsITobin
cleaned it up.. slightly
-
nsITobin
this is tip *-central + patches
-
nsITobin
now
-
WG9s
ok that is now the same error i get has to do with tying to get the audioipc2-client rust package s i think is thunderbird-rust related
-
nsITobin
well good that's progress we both see the same thing.. that's a good sign.. for sanity at least
-
nsITobin
as you know I been encountering pretty much every quirk there is that can block someone building
-
WG9s
nsITobin: are you ring to build seamonkey?
-
nsITobin
atm yes
-
nsITobin
could try Mozilla Thunderthighs
-
nsITobin
i mean
-
nsITobin
Thunderbird
-
WG9s
does it word if you disable-thunderbird-rust?
-
nsITobin
well won't opengpg fail to build without thunderbird rust?
-
nsITobin
pgp
-
nsITobin
whatever it is
-
WG9s
i am trying to build under linux 64 so fails if disalbe-thunderbitd-rust anyway
-
nsITobin
no it was that exchange protocol
-
WG9s
lst i kne seamnkey did not try to build openpgp becuase under 2.53 it can still use the enigmail add-on
-
WG9s
one of the things needed to change for central it needs to build openpgp i think
-
WG9s
but it currently is not so pgp for seamonkey central does not do pgp as far as i know
-
WG9s
becuase it teuite an add-on of a type no longer supported and does not include the opengpg that TB does
-
nsITobin
they did remove a dependancy on the appcore component part that shouldn't have been linked from mailnews
-
WG9s
that is what I thought the current state of this was
-
nsITobin
also configure completed successfully
-
nsITobin
which means m-c is in .. well mozorder
-
nsITobin
and thunderbird busted comm
-
WG9s
one of the three bugs frg landed at the same time the TB build issue began removed a lot of supposeldy unused mailnews code i am thinking pethaps he got a bit carried away in removal
-
WG9s
ibf my build in progress sorrks then when you clone form comm-central speciy changeset
-
WG9s
30c2520e5b0ec3fe26c2c8c69f24352732f37463
-
WG9s
that is last good build before the bustage if you look at treeherder for comm-cdntal
-
nsITobin
I do not buy it's frg's changes unless the patches them selves managled comm-central the repo somehow.. his changes don't affect anyone or rust
-
WG9s
my build still fails so wonder if something changed onm-c that needs a redo syncing
-
nsITobin
I am gonna try a local backout of the last thunderbird rust-sync
-
nsITobin
see what that does
-
nsITobin
oh look configure completed
-
nsITobin
WG9s:
-
nsITobin
now doesn't mean it won't compile error out.. but
-
WG9s
i would suspect one of these patches as causing this
-
WG9s
-
nsITobin
no still saying crap about audioipc2-client
-
nsITobin
unable to update
-
WG9s
and i bet this one since it way audio ipc right in the chck-in comment
-
nsITobin
-
WG9s
it would seem this reuites a comm rust sync which was not performed
-
nsITobin
why does that url have an anchor on it?
-
njsg
-
njsg
nsITobin: the anchor is the start of the commit hash. I chalk it down to github being github
-
njsg
why is the address not /commit/? or is that being output by the build system?
-
nsITobin
by the build system
-
nsITobin
couldn't update
-
nsITobin
yeah i don't understand how rust crates work
-
WG9s
nsITobin: so if you backout the patch for
bug 1910523 that should fix it untll we can get a new sync run it is not an extra sync that is the issue is a missing one that bug requires a sync which never happened.
-
WG9s
wasiest way to fix for now is to backit out
-
WG9s
easiest
-
WG9s
nsITobin: TB folk figured it out and just did a sync so all should be well now
-
nsITobin
oh
-
nsITobin
i was just about to hopefully confirm it was all their fault
-
nsITobin
:P
-
nsITobin
I had to revert the mozilla part and the comm part to return cargo back to its prior state.. guess what I get to do now WG9s lol
-
WG9s
put it all back?
-
nsITobin
YES!
-
nsITobin