-
nsITobin
i need to rebase them against the latest patches but here are my cc patches
code.binaryoutcast.com/thereisonlyx…-patches/src/TRUNK/mzla/comm/series
-
nsITobin
if you care tomman
-
nsITobin
welp guess I am doing
bugzilla.mozilla.org/1912236 on central tomorrow
-
nsITobin
too late tonight
-
nsITobin
Services.tobin.sleep();
-
nsITobin
couldn't sleep so I did this instead.. simplified xulorg's template .. still have to adjust the color of the stolen navigation sidebar
thereisonlyxul.org
-
nsITobin
Good I am happy with thereisonlyxul.org's modifications.. makes it easier to click where I wanna go all in one place
-
nsITobin
good morning #SeaMonkey
-
frg_Away
nsITobin greetings
-
nsITobin
ah hello frg_Away
-
nsITobin
well frg i have VMs to build wip and central and I have a repo to store mah patches in and i have patches to finalize and i modified xulorg's design and front page so i am pretty happy atm
-
nsITobin
cause Real Fun(tm) shall commence shortly, I'm sure.
-
frg_Away
nsITobin given that IanN has not objecxted to
bug 1913787 might wantr to check the Firefox menu. Seems it is using history. I would still keep the sync item in it even if it is unlikely to be activated again in this life.
-
nsITobin
well issue is it was mangled into the go menu
-
frg_Away
would also keep the first four go items
-
nsITobin
this would be resolved with a syncui and backend replacement tho
-
nsITobin
otherwise I am gonna have to split it out to its own function.. and then comment it out and deal with it later
-
nsITobin
personally I'd rather rewrite the previous patch with ifdefs so when/if it gets fixed/replaced/whatever its easy to find
-
frg_Away
maybe into tools? Not a big deal anyway. Or do a seperate part 1 removing it so we cann easier restore it later nor never :)
-
nsITobin
likely navigatorOverlay.js
-
nsITobin
where the menu is defined
-
nsITobin
in the xul file
-
nsITobin
I would THINK
-
nsITobin
splitting it would be better for ANY contengency rewriting the older patch or just backing both out.. ye
-
nsITobin
I will do that
-
WG9s
nsITobin: wondering what it was you thought you needed to do for
bug 1912236. Wanted to make sure I already allied this to suite in the 2.53 gitlab patch queue
wg9s.com/comm-central/patches/seamo…999-port1908725-suite-bustage.patch
-
nsITobin
Ok so you know I have this basic browser buoy window and I use a browser element and thunderbird's MailE10SUtils to manage it so i can have basic navigation .. due to this bug the behavior is it spawns a new process and new main window instead of doing stuff in tthe browser element also doesn't rememeber what it is supposed to do
-
nsITobin
that is how I came across it
-
nsITobin
once I realized it was spawning new processes i knew e10s so i search thunderbird
-
nsITobin
reading that bug was uncanny tho WG9s .. they do not know the history of their own project
-
nsITobin
even those that SHOULD
-
nsITobin
okay so I already have this patch in my working queue WG9s
-
WG9s
so not sure about palemoon but for wuite I needed to add the moz.build change for jsshell wanted to make sure you was that becuase if you do the TB fix in the bug you referenced will reuslt infailured bulding js shell under windows
-
WG9s
trying to make sure you don;t ave to reporduce stuff ws already a solution for
-
nsITobin
what does Pale Moon have anything to do with it?
-
WG9s
i did not know why you thought needed to work on it becuase i had alredy landed the fix on central gitlab patch queue several days erlier
-
nsITobin
i didn't realize you had ported it
-
nsITobin
i guess it isn't enough
-
WG9s
so thought perhaps working on a different project build off of comm-central
-
nsITobin
cause control over browser elements is busted
-
nsITobin
WG9s: at this inital point.. its all the same work
-
WG9s
well se that is wone of the issues where i have only been tasked to make sure the suite still builds and since the UI is terminally broke is hare to test anything
-
nsITobin
getting SM to be able to test stuff requires a working browser element that is controllable as much as an indpendant webrunner does
-
WG9s
OK was confused i figured you either had not seen my patch or was working on a different project (like PaleMooon)
-
nsITobin
I didn't see it
-
WG9s
seems you re past wat i have. do you actually have patces that make the UI work to any extent? if so I would love to add them
-
nsITobin
I only refreshe dthe patches last night and built
-
nsITobin
my state before that was before this landed
-
nsITobin
The current status (before this bug) was SeaMonkey could build.. and launch into an alternatively created main window with devtools and a working --chrome command line handler added to devtool's that is where I locked it off
-
WG9s
so all i did was prot waht seemed to still aplly form the broswer ptches in te initial bug then add stuff not already included and changed in the TB patch and then add the js sell stuff to add the propsys library
-
nsITobin
even without my temp ui just the devtools with chrome cli flag is immensely helpful for testing
-
WG9s
but how do i get to be able to run devtools if the ui does not come up?
-
WG9s
seems you know some magic that i am not aware of
-
nsITobin
well the reason the UI doesn't come up is because we have no functional (and later no) glue that tells the thing what to do with its self
-
WG9s
or have patches applied that i am not aware of
-
nsITobin
WG9s: trenchxulfare
-
nsITobin
i don't recommend it
-
nsITobin
lol
-
nsITobin
WG9s: when we last spoke about this i was just creating and figuring it out
-
nsITobin
but central is secondary to 253 so i don't want to get lost in the wilderness they planeted after bulldozing all the useful stuff
-
WG9s
if you have patches that help and do not break the build send them to me and i will add locally to my build and if they loook good I will try to get frg_Away to approve them
-
nsITobin
I am actively getting them organized and so they can be added on top of current patch queues that is what I was working on past few days
-
nsITobin
I didn't wanna make TOO much of a fuss until i got it good and not changing it every hour lol
-
WG9s
well just saying of they make thinks better on my builds frg_Away would probably love them to be added
-
WG9s
not sure what platforms you test your code on.
-
nsITobin
WG9s: these need rebased and cleaned up and headers fixed but you can see what i gots
code.binaryoutcast.com/thereisonlyx…/brineshrimp-patches/src/TRUNK/mzla
-
nsITobin
central
-
nsITobin
just checked out last night
-
nsITobin
-
nsITobin
more valuable than later version tarballs
-
WG9s
Os so clean them up and fix headers and wend me a zip with all the patches and a series equiv type file to show intended order to apply the patches and I will start incldeding them in my central nightly builds, shich I think would assist in getting the ptches to be accepted
-
nsITobin
WG9s: if you had waited 24 hours the issue of not knowing what I have or have been doing would have been resolved ;)
-
WG9s
and depding on what i think, i resrver the right to add them to the gitlab central quwue without frg_Away approval. Afterall, I do that with my patches.
-
nsITobin
i do my queues with a series file and subdirs so that it can simply be "overlayed" on top of the sm queues and series file contents pasted in
-
nsITobin
so its far easier for others to consume
-
nsITobin
figured it out 2 days ago lol
-
nsITobin
i think its a good idea
-
WG9s
so you have a site somplace with new patches and a series file that I could apply on top of my crap?
-
frg_Away
I usually check in everything which breaks building and leave the others in the queue so that future checkins are easier. Usually it takes much less time to rebase the queue instead of rebasing patches.
-
nsITobin
I don't have em up on a naked http dir no..
-
WG9s
I might prefer to wait till ou think the patches are worthy of landing on the gitlab central zueue
-
nsITobin
-
nsITobin
these should be ok but they need headers revised
-
WG9s
i have 2 states of patches similar. those I apply locally as part of my build script and those I land on gitlab so as to be expected to be added to everyones build
-
nsITobin
null-dll-blocklist is a top patch not for central
-
nsITobin
unless they accept my platform patch which they won't
-
nsITobin
WG9s: I am just copying and pasting back and forth moving my gaurd fake-patches in place.. extrapolated from what frg seems to do
-
nsITobin
i do need a pass to fix the headers tho that needs scripted like you have mentioned WG9s
-
nsITobin
I am seriously getting into this development style i kinda like it
-
WG9s
what is wrong with headers? all i care about in headers is that they have a valid hg header line. A user line that contains a valid contributor name with a valid email address and a valid Date line
-
nsITobin
not all of em do cause not all of em have had an export pass ;)
-
nsITobin
something I am working on
-
WG9s
i don;t like to put things on gitlab that will fail when the convert the Mercuril prop to git thing they do at release time. is fussy about user like without an email and either no or unparsable date format
-
nsITobin
-
nsITobin
-
nsITobin
and do these anywhere
-
nsITobin
-
nsITobin
-
nsITobin
that SHOULD get you access to spawn any window from commandline as well as an attached browser console by normal means
-
WG9s
otheres seem to care about NOde ID and Parent ehaders but as they do not break the port HG stuff to git thing I don;t really care.
-
nsITobin
-
nsITobin
so it can run
-
nsITobin
and not crash
-
nsITobin
the rest is a mess
-
nsITobin
or busted now with that bug
-
nsITobin
WG9s: I am not sure it actually matters in mq
-
nsITobin
the node id and parent headers
-
nsITobin
but i bet bugzilla likes em
-
WG9s
exactly frg_Away seems to like to change them my feeling is no one cares
-
nsITobin
but I will do them as you guys want because the idea of contributing is not making extra work lol
-
WG9s
because they are likely to depend on other work in local queue so incorrect anyway
-
WG9s
for nayonone other than you (or me on mine)
-
nsITobin
I try very hard not to make those mistakes.. but no promises ;)
-
nsITobin
WG9s: this little patch repo
-
nsITobin
also has another secret
-
nsITobin
-
nsITobin
it's unfinished but helps
-
nsITobin
heh need to update the bundle baseurl
-
frg_Away
ids are recalculated during qrefresh. Doesn't matter. I only check ocassionally when I backport with the original bug that I have the reght revision. Afterwards new or old doesn't matter.
-
nsITobin
my intenion with this script is to basically automate a simple way the task of getting wip (and 115) or central, running bootstrap in 115 or central (in central's case change the origin url to mozilla and pull) and then patch.. and after ALL THAT.. Then apply my patches ;)
-
nsITobin
I also want to attempt to figure out a way to rewrite the visual studio finding logic to pick up the cached 2019 version that 115 downloads if it is there otherwise exclude it and download and run vswhere with a predefined configuration
-
nsITobin
so that the entire process is minimal hassle even if it will take quite some time
-
nsITobin
among other patches both 253 and central and other BinOC things too like an operating system lol
-
nsITobin
I must do more.
-
nsITobin
WG9s: there is a simple cheat to ns*App.cpp
-
nsITobin
copypasta
-
nsITobin
and change Thunderbird to SeaMonkey
-
WG9s
nsITobin: but would be great f we were not at cross purposes. WOuld need to treat the central hg repo plus the gitlab seamonkey-central-patchs repo as the base so that we are wriing patches based on te same base.
-
WG9s
and if you think that might work i will try to get you patches landed on gitlab based only on my approval
-
WG9s
so like if i think this breaks nothing and maes tings better i sill wuickly approve and add
-
WG9s
we seems to both have the same goal so no reason to not be coorperating
-
nsITobin
WG9s: does it make sense to still consider central as future seamonkey 253?
-
nsITobin
or maybe I am misunderstanding
-
WG9s
frg has gen up on central but maybe not adverse to geting it ot m-c68 esr
-
WG9s
so still possible issue is he stillwants to keep wupporting legacy add-ons
-
nsITobin
WG9s: how can one build esr68 tho i couldn't get it to do it because I'd need to manually solve deps because taskcluster no longer cares or provides for older codebases
-
WG9s
and i think many don;t get that that is a huge security issue. is an API that add-ons are supposed ot use but nothing prevents a legacy add-on to bypass the apis and just access global variables to make completely arbitrary unsupported changes to the code.
-
nsITobin
WG9s: there is no reason to believe one cannot have XUL Extensions at ESR68 just need some reverts it just may not be the full spectrum of restartful toolkit extensions.. they would be bootstrapped unless changes were made otherwise.. THIS is an area I know very well the jump platform upgrade and revert thing
-
frg_Away
esr 68 makes no sense. missing tons of later js stuff and legacy add-on support is so gutted it would basically all add-ons to be changed.
-
WG9s
so the mozilla firefox pi\eople tried to do this by only allowing add-ons on AMO that did not do this, but then nothing preventing add-ons form other sources
-
WG9s
other than a warning
-
WG9s
so have to overrride that to tet ad blocking to use and once you do then anyone else with a malicious add-on is also permitted
-
nsITobin
WG9s: I disagree with you on the security issue of having powerful application modifing extensions and here's why.. Properly informed and educated users with approerate education and informational resources are more than capable of deciding if they want or should run packaged app hacks and the consiquences if it goes wrong.. This is what killed the technology.. lack of proper user education .. and a willingnes to have a powerful open extension system
-
nsITobin
open by default in the 2010s when most don't need much as "proven" by webextensions.. it should be there but opt in.. a power user feature for power user modifications.. but this again is only a problem because of Mozilla mistakes and sabotaging by dumb decisions the fostering and curation of this technology and ecosystem
-
nsITobin
also i want them too LOL but i also want binary xpcom extensions on top ;)
-
WG9s
a whitelist of add-ons that are permitted without having to get them working by allowing all add-ons this priv woud have been way more appropriate.
-
WG9s
they tried this but not sure what happened and decided to just not allow legacy add-ons at al
-
nsITobin
I'd be fine with babymode computing as the default if I had a quick and effective way to make it go away so I cna get back to what I was doing that is my problem with modern security
-
nsITobin
Modern Security is drenched in the assumption that I as the user do not know better than the machine
-
nsITobin
user.. OPERATOR
-
nsITobin
System Operator
-
WG9s
I think your (and my) issues are the same people wtih no idea bout security trying to dictate what is best for security reasons.
-
nsITobin
if I question it.. maybe I can make the decision my self?
-
nsITobin
Privacy, Security, and general risk assessment can never be one fits all
-
WG9s
nsOreason i said go via me for now is I am authozed to change gitlab central aptch queue and am not sure you are..
-
nsITobin
if I want to run Windows XP as NT AUTHORITY\SYSTEM on an unfirewalled open internet connection.. I am prepared to mitigate or face the consiquences of that action. Even if I admit its kinda stupid.
-
nsITobin
and make a very huge understatement ;)
-
WG9s
but I still think betr if we work together
-
WG9s
-
nsITobin
I don't want anymore access to anything than you and the others feel is approrate and required to faciliate activities.
-
WG9s
nsITobin: not up to me
-
nsITobin
i don't think I need anything currently and i might mess it up as I am still leanring and forming my own workflows based upon em
-
nsITobin
except to document what and how and whatever i am doing better
-
nsITobin
maybe start a blog
-
nsITobin
LOL
-
nsITobin
so its all.. well what is tobin doing *looks* ah well this and this i want this is crap and this needs to be rewritten AND WHY are there still BinOC patches lol
-
WG9s
just sayin if you ork from same base as me easier fo rme to advocate for you patch if you anet me to build using teh current hg and ignote the gitlab patch queue and use your's instead (which is what I think you said) not so easy fro me to help you here
-
nsITobin
I can stay in sync
-
nsITobin
I just didn't do that initally cause its INSANE on 253 atm
-
WG9s
I knew you wul understand
-
nsITobin
try forming it as an order
-
nsITobin
;)
-
WG9s
if you work on the same base as me i can easiy test your patches on allplatroms
-
nsITobin
seriously yeah I didn't mean to split off but that last bustage and this one has kinda thown a wrench in my central queue
-
nsITobin
cause I was working on prelem es conversion by hand
-
nsITobin
but changed gears becuase that is gonna be a time sink and i need to get stuff where you guys can easily access it instead of throwing links constantly to adhoc files
-
WG9s
nsITobin: not making it an order just saying fi not forking is wasier to get it landed frg_Away wirites patches for central to include m bustage fixes. he is a way behind of this becuase of his other issues, but is now back to all my pending bustage patches will end up on comm and mozilla central so would be better if you dealt with thm now
-
WG9s
but also if one of tem looks completely bogus let me knwo and i might remove it if you have a better replacement
-
nsITobin
well I got the VM for central up
-
WG9s
also shuld havw been py8ing for attention. my requiremnt meant did not require a rull UI but required at least a small window with an X choice to kill the app
-
WG9s
seems that had not worked in a log time. I should ahve been apytin more attention.
-
nsITobin
WG9s: I been chaotic with false starts for over 2 years now.. i don't blame you for not paying much attention until stuff materalially starts happening
-
nsITobin
the ONE damned thing that is annoying about working with patches and it isn't mq's fault is popping off anything configure or a configdep means i have to wait an hour
-
nsITobin
kicked off with a whole code transplant from Thunderbird's nsMailApp.cpp to SeaMonkey's nsSuiteApp.cpp (with a few strings replaced)
-
nsITobin
any bets if it will work, work as intended, or fail
-
WG9s
i have no ide
-
WG9s
idea IanN was last to look at this code from the suite perspective
-
WG9s
frg_Away: prefers changes to be bug by bug changes and not a wholesale replace, just sayin
-
WG9s
nsITobin: besided ust replacing one .cpp file with a newer version without looking at the changes this actually had done on central in defined version changed other files that your just replace this does not do so this is more likely to cause more than it fixes. Just my opinion.
-
nsITobin
WG9s: if i reverse the clang style changes it will be effectively the bug
-
WG9s
oh ok then fine
-
nsITobin
xre bootstrap made ns*App.cpp almsot identical across the board
-
WG9s
but still check with frg_Away sometimes he is fine with jut doing the style changes other times he prefers dong the changes with out the style changes
-
nsITobin
indeed
-
nsITobin
the downside with --jsconsole and --chrome vs entry points from Buoy is you only have the life of that one window and since everything is busted you can't do much else.. that is why the minimal window I created as a substute entry point for SuiteGlue effectively and as an isolated working (theoretically) thing can let you fix what you want if you want or at least pull something up in the browser element
-
nsITobin
WG9s: its a crime you won't find ANY of this outside deep in the shitpile of the Pale Moon forum
-
nsITobin
or this channel's log
-
nsITobin
the real js console was more useful for this as an ancor
-
nsITobin
but that is long gone
-
nsITobin
WG9s: did you add the followups? maybe that's the issue for the difference
-
nsITobin
yep I think that is it
-
nsITobin
why didn't they squash this before landing proper
-
nsITobin
it is my estmimation the last real codebase to have something akin to a XUL platform would be esr102 with heavy reverts and retroengineering.. beyond that there just isn't enough mozilla left in Mozilla beyond the rendering engine and accessing it is a pain in the ass these days but I will find a way for one project or another
-
nsITobin
I wonder how bad thunderbird reacted to this.. i bet there were half a dozen main windows open all not right
-
nsITobin
WG9s: confirmed
-
nsITobin
needed followup patches
-
WG9s
nsITobin: so if you do those either one bug at at ime (i am not a sticler if more that one patch in a bug (ust try8ing to satisfy frg_Away requirements.
-
frg_Away
IanNs too :)
-
nsITobin
it was easy to miss the last follow up landed a day later and without having anything to actually display and see how could you have known anything..
-
frg_Away
When I did the library replacement it was almost impossible to do this bug based but that was/is one of the few exceptions.
-
nsITobin
frg_Away: do you care if nsBrowserApp is clangformatted?
-
nsITobin
err
-
nsITobin
suite app
-
frg_Away
I usually do manual formatting. IanN is sometimes allergic when it comes to my reformatting in patches so I try to not overdo it.
-
frg_Away
Anyway one day we need to do it for all files.
-
nsITobin
see while clangformatting moves the braces up like I like it also makes all the function arguments a messy list instead of listed out unless just a few
-
nsITobin
well from experience thunderbird's nsMailApp as-is with just a few string changes is a very effective and almost universal match for anything the suite is doing
-
nsITobin
especially now that most of the code was shipped off to xre bootstrap
-
nsITobin
in UXP everything was ultimently based on Basilisk's refernece when such a cheap hack would do ns*App, shellservice, other things.. aboutRedirector tho.. Thunderbird
-
frg_Away
nsITobin I usually move the braces up and leave the function alone. Current code looks all messy in mozilla-central
-
nsITobin
frg_Away: which formatting do you prefer? netscape, late mozilla, or clang
-
nsITobin
not that I am the greatest at ahearing to it I prefer to code in what I deem a updated netscape style.. which is different than late mozilla tho some earlier styles are still present in this PHP code
-
nsITobin
-
nsITobin
i need to fix a lot of alignment
-
nsITobin
.. mass renames
-
nsITobin
how mozilla of me
-
frg_Away
nsobin probably like a mixup too. Around mozilla 68 was ok so maybe clang 7 8 . The late formatting is bad
-
nsITobin
i'd seriously consider doing some pathfinding with esr68 but I can't build the bloody thing closest thing to it i can build is..
-
nsITobin
wait for it
-
nsITobin
SeaMonkey ;)
-
nsITobin
of course you know frg_Away .. having a patchqueue would have been invaluable to how I managed to achieve vNext 1 and 2 on Pale Moon
-
nsITobin
it was lots of branching long recommit times and then reapplying everything and doing a little more and over and over again up every mozilla version.. just aiming for and you guessed it.. simple buildabiliy first and foremost
-
frg_Away
esr68 would probably build with an old rust version and clang.
-
nsITobin
how do I downgrade clang
-
nsITobin
i downgraded rust but it failed
-
nsITobin
it's bugging me cause the xp people can do it
-
nsITobin
I shall not have the capabilities of Windows XP users exceed mine! lol
-
frg_Away
nsITobin you now and then grab artifacts from taskcluster. I thinki have clang 9 and 14 for Windows.
-
nsITobin
are they still up there?
-
frg_Away
That s how we cross compile for macOS
-
frg_Away
but basically same for Windows.
-
WG9s
i have some old slng versions tar files tha you can install in ~/.mozbuild
-
WG9s
and then specify them in your .mozconfig
-
nsITobin
WG9s: we should put it up on files dot xulorg
-
nsITobin
for sure
-
frg_Away
nsITobin not sure. Doubt it.
-
WG9s
if have clang 11 and 13-17 in additon to the version 18 that mach bootstrap installs
-
frg_Away
-
frg_Away
so you can't find them via this route.
-
WG9s
then in my bmozconbfig at top of the file I export CLANG=$home/mozbuild/clang and then do everytong else like CC CXX etc uisng CLANG
-
nsITobin
right i was searching for taskcluster
-
nsITobin
re-bookmarked
-
WG9s
if you can;t find what you need elsehere i can email you the tar file
-
nsITobin
WG9s: I sent you creds to upload stuff to gallium if needed
-
nsITobin
please don't fill the disk the logbot doesn't like that ;)
-
frg_Away
gitlab wip updated. If anyone misses nsIDOMSVGLength aka
bug 1435138 let me know.
-
frg_Away
will move it over to 2.53 next too.
-
nsITobin
frg_Away: trying to think what outside the machinery of website rendering scope would use it for
-
frg_Away
nsITobin and then there are also sw´vg web interfaces too.
-
nsITobin
well i just dunno what use svg manipulation has in an extension
-
nsITobin
unless you wanted to make an svg creator
-
frg_Away
-
frg_Away
nsITobin anything which every used it is probably so dead no one remembers it. And new stuff can use the regular interfaces so no loss I assume.
-
nsITobin
well of the popular sub-thousand extensions in my head .. but in a mist of sorts.. i can't recall any of em being anything svg related
-
frg_Away
my small brain neither.
-
nsITobin
firefox extension mist is not a typical brain condition i trust