From e7a0751a2cbc85346465da3b26c5de0bedef598e Mon Sep 17 00:00:00 2001 From: Beq Date: Sun, 17 Aug 2025 21:15:14 +0100 Subject: [PATCH 1/5] Try g++-14 on the slow older runners :-( --- .github/workflows/build_viewer.yml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/build_viewer.yml b/.github/workflows/build_viewer.yml index 5c8697fc82..243145dc37 100644 --- a/.github/workflows/build_viewer.yml +++ b/.github/workflows/build_viewer.yml @@ -35,12 +35,9 @@ jobs: build_matrix: strategy: matrix: - os: [macos-15,ubuntu-24.04,windows-2022] + os: [macos-15,ubuntu-22.04,windows-2022] grid: [sl,os] variant: [regular, avx] - include: - - os: ubuntu-24.04 - container_image: ubuntu:22.04 # only Linux uses a container runs-on: ${{ matrix.os }} container: ${{ matrix.container_image }} outputs: @@ -116,6 +113,7 @@ jobs: echo "CC=gcc-14" >> $GITHUB_ENV echo "CXX=g++-14" >> $GITHUB_ENV + - name: Setup rclone and download the folder uses: beqjanus/setup-rclone@main with: From 70768b815f373dd98df64187656f90e247fc9136 Mon Sep 17 00:00:00 2001 From: Beq Date: Sun, 17 Aug 2025 21:38:24 +0100 Subject: [PATCH 2/5] Final permutation test. try gcc-13 and hope. --- .github/workflows/build_viewer.yml | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/.github/workflows/build_viewer.yml b/.github/workflows/build_viewer.yml index 50919c28ad..790502eb8a 100644 --- a/.github/workflows/build_viewer.yml +++ b/.github/workflows/build_viewer.yml @@ -125,18 +125,16 @@ jobs: with: swap-storage: false - - name: Install GCC-14 + - name: Install GCC-13 if: runner.os == 'Linux' run: | sudo apt-get update - sudo apt-get install -y software-properties-common sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test - sudo apt-get update - sudo apt-get install -y gcc-14 g++-14 - echo "CC=gcc-14" >> $GITHUB_ENV - echo "CXX=g++-14" >> $GITHUB_ENV - - + sudo apt-get install -y gcc-13 g++-13 + sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-13 130 \ + --slave /usr/bin/g++ g++ /usr/bin/g++-13 + echo "CC=gcc-13" >> $GITHUB_ENV + echo "CXX=g++-13" >> $GITHUB_ENV - name: Setup rclone and download the folder uses: beqjanus/setup-rclone@main with: From 1227e538b1006c0b4f6d337e10480c2057b2331b Mon Sep 17 00:00:00 2001 From: Ansariel Date: Tue, 19 Aug 2025 13:34:08 +0200 Subject: [PATCH 3/5] Fix the particle editor --- indra/newview/particleeditor.cpp | 61 ++++-------- indra/newview/particleeditor.h | 164 +++++++++++++++---------------- 2 files changed, 103 insertions(+), 122 deletions(-) diff --git a/indra/newview/particleeditor.cpp b/indra/newview/particleeditor.cpp index 5f8e591dc7..70fab5970d 100644 --- a/indra/newview/particleeditor.cpp +++ b/indra/newview/particleeditor.cpp @@ -22,8 +22,6 @@ #include "llviewerprecompiledheaders.h" #include "particleeditor.h" -#include - #include "llagent.h" #include "llappviewer.h" #include "llcheckboxctrl.h" @@ -47,10 +45,7 @@ #include "llwindow.h" #include "llviewerassetupload.h" -ParticleEditor::ParticleEditor(const LLSD& key) -: LLFloater(key), - mObject(0), - mParticleScriptInventoryItem(0) +ParticleEditor::ParticleEditor(const LLSD& key) : LLFloater(key) { mPatternMap["drop"] = LLPartSysData::LL_PART_SRC_PATTERN_DROP; mPatternMap["explode"] = LLPartSysData::LL_PART_SRC_PATTERN_EXPLODE; @@ -325,11 +320,11 @@ void ParticleEditor::updateUI() bool targetLinear = mTargetLinearCheckBox->getValue(); bool interpolateColor = mInterpolateColorCheckBox->getValue(); bool interpolateScale = mInterpolateScaleCheckBox->getValue(); - bool targetEnabled = targetLinear | (mTargetPositionCheckBox->getValue().asBoolean() ? true : false); + bool targetEnabled = targetLinear || (mTargetPositionCheckBox->getValue().asBoolean() ? true : false); - mBurstRadiusSpinner->setEnabled(!(targetLinear | (mFollowSourceCheckBox->getValue().asBoolean() ? true : false) | dropPattern)); - mBurstSpeedMinSpinner->setEnabled(!(targetLinear | dropPattern)); - mBurstSpeedMaxSpinner->setEnabled(!(targetLinear | dropPattern)); + mBurstRadiusSpinner->setEnabled(!targetLinear && !(mFollowSourceCheckBox->getValue().asBoolean() ? true : false) && !dropPattern); + mBurstSpeedMinSpinner->setEnabled(!targetLinear && !dropPattern); + mBurstSpeedMaxSpinner->setEnabled(!targetLinear && !dropPattern); // disabling a color swatch does nothing visually, so we also set alpha LLColor4 endColor = mEndColorSelector->get(); @@ -355,8 +350,8 @@ void ParticleEditor::updateUI() mOmegaYSpinner->setEnabled(!targetLinear); mOmegaZSpinner->setEnabled(!targetLinear); - mAngleBeginSpinner->setEnabled(!(explodePattern | dropPattern)); - mAngleEndSpinner->setEnabled(!(explodePattern | dropPattern)); + mAngleBeginSpinner->setEnabled(!explodePattern && !dropPattern); + mAngleEndSpinner->setEnabled(!explodePattern && !dropPattern); } void ParticleEditor::onClearTargetButtonClicked() @@ -369,15 +364,13 @@ void ParticleEditor::onTargetPickerButtonClicked() { mPickTargetButton->setToggleState(true); mPickTargetButton->setEnabled(false); - startPicking(this); + startPicking(); } // inspired by the LLFloaterReporter object picker -// static -void ParticleEditor::startPicking(void* userdata) +void ParticleEditor::startPicking() { - ParticleEditor* self = (ParticleEditor*) userdata; - LLToolObjPicker::getInstance()->setExitCallback(ParticleEditor::onTargetPicked, self); + LLToolObjPicker::getInstance()->setExitCallback(ParticleEditor::onTargetPicked, this); LLToolMgr::getInstance()->setTransientTool(LLToolObjPicker::getInstance()); } @@ -386,14 +379,12 @@ void ParticleEditor::onTargetPicked(void* userdata) { ParticleEditor* self = (ParticleEditor*)userdata; - LLUUID picked = LLToolObjPicker::getInstance()->getObjectID(); - LLToolMgr::getInstance()->clearTransientTool(); self->mPickTargetButton->setEnabled(true); self->mPickTargetButton->setToggleState(false); - if (picked.notNull()) + if (LLUUID picked = LLToolObjPicker::getInstance()->getObjectID(); picked.notNull()) { self->mTargetKeyInput->setValue(picked.asString()); self->onParameterChange(); @@ -522,8 +513,7 @@ default\n\ void ParticleEditor::onCopyButtonClicked() { - std::string script = createScript(); - if (!script.empty()) + if (std::string script = createScript(); !script.empty()) { getWindow()->copyTextToClipboard(utf8str_to_wstring(script)); LLNotificationsUtil::add("ParticleScriptCopiedToClipboard"); @@ -608,9 +598,7 @@ void ParticleEditor::callbackReturned(const LLUUID& inventoryItemID) gInventory.notifyObservers(); //caps import - std::string url = gAgent.getRegionCapability("UpdateScriptAgent"); - - if (!url.empty()) + if (std::string url = gAgent.getRegionCapability("UpdateScriptAgent"); !url.empty()) { std::string script = createScript(); @@ -633,30 +621,23 @@ void ParticleEditor::callbackReturned(const LLUUID& inventoryItemID) void ParticleEditor::scriptInjectReturned() { setCanClose(true); + mMainPanel->setEnabled(true); // play it safe, because some time may have passed - LLViewerObject* object = gObjectList.findObject(mObject->getID()); - if (!object || mObject->isDead()) + if (LLViewerObject* object = gObjectList.findObject(mObject->getID()); object && mObject&& !mObject->isDead()) + { + mObject->saveScript(mParticleScriptInventoryItem, true, false); + LLNotificationsUtil::add("ParticleScriptInjected"); + } + else { LL_WARNS() << "Can't inject script - object is dead or went away!" << LL_ENDL; - mMainPanel->setEnabled(true); - return; } - - mObject->saveScript(mParticleScriptInventoryItem, true, false); - LLNotificationsUtil::add("ParticleScriptInjected"); - - delete this; } // ---------------------------------- Callbacks ---------------------------------- -ParticleScriptCreationCallback::ParticleScriptCreationCallback(ParticleEditor* editor) -{ - mEditor = editor; -} - -ParticleScriptCreationCallback::~ParticleScriptCreationCallback() +ParticleScriptCreationCallback::ParticleScriptCreationCallback(ParticleEditor* editor) : mEditor(editor) { } diff --git a/indra/newview/particleeditor.h b/indra/newview/particleeditor.h index d0865d8644..c930866822 100644 --- a/indra/newview/particleeditor.h +++ b/indra/newview/particleeditor.h @@ -43,117 +43,117 @@ class ParticleEditor : public LLFloater friend class ParticleScriptCreationCallback; friend class ParticleScriptUploadResponder; - public: - ParticleEditor(const LLSD& key); - ~ParticleEditor(); +public: + ParticleEditor(const LLSD& key); + ~ParticleEditor(); - bool postBuild() override; + bool postBuild() override; - void setObject(LLViewerObject* objectp); - void scriptInjectReturned( ); + void setObject(LLViewerObject* objectp); + void scriptInjectReturned(); - protected: - void clearParticles(); - void updateParticles(); - void updateUI(); +protected: + void clearParticles(); + void updateParticles(); + void updateUI(); - std::string createScript(); - void createScriptInventoryItem(LLUUID categoryID); + std::string createScript(); + void createScriptInventoryItem(LLUUID categoryID); - void onParameterChange(); - void onCopyButtonClicked(); - void onInjectButtonClicked(); + void onParameterChange(); + void onCopyButtonClicked(); + void onInjectButtonClicked(); - void onClearTargetButtonClicked(); - void onTargetPickerButtonClicked(); - static void startPicking(void* userdata); - static void onTargetPicked(void* userdata); + void onClearTargetButtonClicked(); + void onTargetPickerButtonClicked(); + void startPicking(); + static void onTargetPicked(void* userdata); - void callbackReturned(const LLUUID& inv_item); + void callbackReturned(const LLUUID& inv_item); - std::string lslVector(F32 x, F32 y, F32 z); - std::string lslColor(const LLColor4& color); + std::string lslVector(F32 x, F32 y, F32 z); + std::string lslColor(const LLColor4& color); - LLViewerObject* mObject; - LLViewerTexture* mTexture; - LLViewerInventoryItem* mParticleScriptInventoryItem; + LLViewerObject* mObject{ nullptr }; + LLViewerTexture* mTexture{ nullptr }; + LLViewerInventoryItem* mParticleScriptInventoryItem{ nullptr }; - LLViewerTexture* mDefaultParticleTexture; + LLViewerTexture* mDefaultParticleTexture{ nullptr }; - LLPartSysData mParticles; + LLPartSysData mParticles; - std::map mPatternMap; - std::map mScriptPatternMap; + std::map mPatternMap; + std::map mScriptPatternMap; - std::map mBlendMap; - std::map mScriptBlendMap; + std::map mBlendMap; + std::map mScriptBlendMap; - LLPanel* mMainPanel; + LLPanel* mMainPanel{ nullptr }; - LLComboBox* mPatternTypeCombo; - LLTextureCtrl* mTexturePicker; + LLComboBox* mPatternTypeCombo{ nullptr }; + LLTextureCtrl* mTexturePicker{ nullptr }; - LLSpinCtrl* mBurstRateSpinner; - LLSpinCtrl* mBurstCountSpinner; - LLSpinCtrl* mBurstRadiusSpinner; - LLSpinCtrl* mAngleBeginSpinner; - LLSpinCtrl* mAngleEndSpinner; - LLSpinCtrl* mBurstSpeedMinSpinner; - LLSpinCtrl* mBurstSpeedMaxSpinner; - LLSpinCtrl* mStartAlphaSpinner; - LLSpinCtrl* mEndAlphaSpinner; - LLSpinCtrl* mScaleStartXSpinner; - LLSpinCtrl* mScaleStartYSpinner; - LLSpinCtrl* mScaleEndXSpinner; - LLSpinCtrl* mScaleEndYSpinner; - LLSpinCtrl* mSourceMaxAgeSpinner; - LLSpinCtrl* mParticlesMaxAgeSpinner; - LLSpinCtrl* mStartGlowSpinner; - LLSpinCtrl* mEndGlowSpinner; + LLSpinCtrl* mBurstRateSpinner{ nullptr }; + LLSpinCtrl* mBurstCountSpinner{ nullptr }; + LLSpinCtrl* mBurstRadiusSpinner{ nullptr }; + LLSpinCtrl* mAngleBeginSpinner{ nullptr }; + LLSpinCtrl* mAngleEndSpinner{ nullptr }; + LLSpinCtrl* mBurstSpeedMinSpinner{ nullptr }; + LLSpinCtrl* mBurstSpeedMaxSpinner{ nullptr }; + LLSpinCtrl* mStartAlphaSpinner{ nullptr }; + LLSpinCtrl* mEndAlphaSpinner{ nullptr }; + LLSpinCtrl* mScaleStartXSpinner{ nullptr }; + LLSpinCtrl* mScaleStartYSpinner{ nullptr }; + LLSpinCtrl* mScaleEndXSpinner{ nullptr }; + LLSpinCtrl* mScaleEndYSpinner{ nullptr }; + LLSpinCtrl* mSourceMaxAgeSpinner{ nullptr }; + LLSpinCtrl* mParticlesMaxAgeSpinner{ nullptr }; + LLSpinCtrl* mStartGlowSpinner{ nullptr }; + LLSpinCtrl* mEndGlowSpinner{ nullptr }; - LLComboBox* mBlendFuncSrcCombo; - LLComboBox* mBlendFuncDestCombo; + LLComboBox* mBlendFuncSrcCombo{ nullptr }; + LLComboBox* mBlendFuncDestCombo{ nullptr }; - LLCheckBoxCtrl* mBounceCheckBox; - LLCheckBoxCtrl* mEmissiveCheckBox; - LLCheckBoxCtrl* mFollowSourceCheckBox; - LLCheckBoxCtrl* mFollowVelocityCheckBox; - LLCheckBoxCtrl* mInterpolateColorCheckBox; - LLCheckBoxCtrl* mInterpolateScaleCheckBox; - LLCheckBoxCtrl* mTargetPositionCheckBox; - LLCheckBoxCtrl* mTargetLinearCheckBox; - LLCheckBoxCtrl* mWindCheckBox; - LLCheckBoxCtrl* mRibbonCheckBox; + LLCheckBoxCtrl* mBounceCheckBox{ nullptr }; + LLCheckBoxCtrl* mEmissiveCheckBox{ nullptr }; + LLCheckBoxCtrl* mFollowSourceCheckBox{ nullptr }; + LLCheckBoxCtrl* mFollowVelocityCheckBox{ nullptr }; + LLCheckBoxCtrl* mInterpolateColorCheckBox{ nullptr }; + LLCheckBoxCtrl* mInterpolateScaleCheckBox{ nullptr }; + LLCheckBoxCtrl* mTargetPositionCheckBox{ nullptr }; + LLCheckBoxCtrl* mTargetLinearCheckBox{ nullptr }; + LLCheckBoxCtrl* mWindCheckBox{ nullptr }; + LLCheckBoxCtrl* mRibbonCheckBox{ nullptr }; - LLLineEditor* mTargetKeyInput; - LLButton* mClearTargetButton; - LLButton* mPickTargetButton; + LLLineEditor* mTargetKeyInput{ nullptr }; + LLButton* mClearTargetButton{ nullptr }; + LLButton* mPickTargetButton{ nullptr }; - LLSpinCtrl* mAcellerationXSpinner; - LLSpinCtrl* mAcellerationYSpinner; - LLSpinCtrl* mAcellerationZSpinner; + LLSpinCtrl* mAcellerationXSpinner{ nullptr }; + LLSpinCtrl* mAcellerationYSpinner{ nullptr }; + LLSpinCtrl* mAcellerationZSpinner{ nullptr }; - LLSpinCtrl* mOmegaXSpinner; - LLSpinCtrl* mOmegaYSpinner; - LLSpinCtrl* mOmegaZSpinner; + LLSpinCtrl* mOmegaXSpinner{ nullptr }; + LLSpinCtrl* mOmegaYSpinner{ nullptr }; + LLSpinCtrl* mOmegaZSpinner{ nullptr }; - LLColorSwatchCtrl* mStartColorSelector; - LLColorSwatchCtrl* mEndColorSelector; + LLColorSwatchCtrl* mStartColorSelector{ nullptr }; + LLColorSwatchCtrl* mEndColorSelector{ nullptr }; - LLButton* mCopyToLSLButton; - LLButton* mInjectScriptButton; + LLButton* mCopyToLSLButton{ nullptr }; + LLButton* mInjectScriptButton{ nullptr }; }; class ParticleScriptCreationCallback : public LLInventoryCallback { - public: - ParticleScriptCreationCallback(ParticleEditor* editor); - void fire(const LLUUID& inventoryItem); +public: + ParticleScriptCreationCallback(ParticleEditor* editor); + void fire(const LLUUID& inventoryItem); - protected: - ~ParticleScriptCreationCallback(); +protected: + ~ParticleScriptCreationCallback() = default; - ParticleEditor* mEditor; + ParticleEditor* mEditor; }; #endif // PARTICLEEDITOR_H From 1be2b04093a1a68378e85439c72834c05181ef23 Mon Sep 17 00:00:00 2001 From: Hecklezz Date: Wed, 20 Aug 2025 03:10:50 +1000 Subject: [PATCH 4/5] Added debug setting to hide help button from floaters --- indra/llui/llfloater.cpp | 6 ++++-- indra/newview/app_settings/settings.xml | 11 +++++++++++ 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/indra/llui/llfloater.cpp b/indra/llui/llfloater.cpp index c33bf14354..548ee2037d 100644 --- a/indra/llui/llfloater.cpp +++ b/indra/llui/llfloater.cpp @@ -338,8 +338,10 @@ void LLFloater::initFloater(const Params& p) // Help button: '?' //SL-14050 Disable all Help question marks - // Nope! - mButtonsEnabled[BUTTON_HELP] = !mHelpTopic.empty();// false; + // Disable the help button only if the debug setting is on + static LLUICachedControl hide_help_buttons ("FSHideHelpButtons", false); + mButtonsEnabled[BUTTON_HELP] = !mHelpTopic.empty() && !hide_help_buttons;// false; + // // Minimize button only for top draggers if ( !mDragOnLeft && mCanMinimize ) diff --git a/indra/newview/app_settings/settings.xml b/indra/newview/app_settings/settings.xml index e8d6fcdc6a..7531ea03c2 100644 --- a/indra/newview/app_settings/settings.xml +++ b/indra/newview/app_settings/settings.xml @@ -26762,5 +26762,16 @@ Change of this parameter will affect the layout of buttons in notification toast 0 + FSHideHelpButtons + + Comment + When enabled, hides the help button from floaters (requires restart) + Persist + 1 + Type + Boolean + Value + 0 + From 489d1bb0440e37c1b217c6157fba4dcd620ab225 Mon Sep 17 00:00:00 2001 From: Beq Date: Fri, 22 Aug 2025 22:43:24 +0100 Subject: [PATCH 5/5] Add missing GLTF filter to mesh model loader on Linux --- indra/newview/llfilepicker.cpp | 19 ++++++++++++++++++- .../newview/skins/default/xui/en/strings.xml | 1 + 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/indra/newview/llfilepicker.cpp b/indra/newview/llfilepicker.cpp index b81d74cc9f..e2f2bdf5fc 100644 --- a/indra/newview/llfilepicker.cpp +++ b/indra/newview/llfilepicker.cpp @@ -1455,6 +1455,20 @@ static std::string add_collada_filter_to_gtkchooser(GtkWindow *picker) LLTrans::getString("collada_files") + " (*.dae)"); } +// migrate to GLTF support +static std::string add_model_filter_to_gtkchooser(GtkWindow *picker) +{ +// "Model files (*.dae, *.gltf, *.glb)" + GtkFileFilter *gfilter = gtk_file_filter_new(); + gtk_file_filter_add_pattern(gfilter, "*.dae"); + gtk_file_filter_add_pattern(gfilter, "*.gltf"); + gtk_file_filter_add_pattern(gfilter, "*.glb"); + std::string filtername = LLTrans::getString("model_files") + " (*.dae; *.gltf; *.glb)"; + add_common_filters_to_gtkchooser(gfilter, picker, filtername); + return filtername; +} +// + static std::string add_imageload_filter_to_gtkchooser(GtkWindow *picker) { GtkFileFilter *gfilter = gtk_file_filter_new(); @@ -1676,7 +1690,7 @@ bool LLFilePicker::getOpenFile( ELoadFilter filter, bool blocking ) filtername = dead_code_should_blow_up_here(picker); break; case FFLOAD_COLLADA: - filtername = add_collada_filter_to_gtkchooser(picker); + filtername = add_model_filter_to_gtkchooser(picker); break; case FFLOAD_IMAGE: filtername = add_imageload_filter_to_gtkchooser(picker); @@ -1936,6 +1950,9 @@ bool LLFilePicker::openFileDialog( int32_t filter, bool blocking, EType aType ) file_dialog_filter = "*.raw"; break; case FFLOAD_MODEL: + file_type = "model_files"; + file_dialog_filter = "*.{dae,gltf,glb}"; + break; case FFLOAD_COLLADA: file_type = "collada_files"; file_dialog_filter = "*.dae"; diff --git a/indra/newview/skins/default/xui/en/strings.xml b/indra/newview/skins/default/xui/en/strings.xml index acae761190..4015351fb5 100644 --- a/indra/newview/skins/default/xui/en/strings.xml +++ b/indra/newview/skins/default/xui/en/strings.xml @@ -663,6 +663,7 @@ https://www.firestormviewer.org/support for help fixing this problem. Dictionaries Object Backups COLLADA Models + Mesh Models Comma separated values Recompile