** Changed in: ubuntustudio
Status: Fix Committed => Fix Released
** Changed in: ubuntustudio-menu-add
Status: Fix Committed => Fix Released
** Changed in: ubuntu
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Ubuntu
All bugs have since been ironed out (details in the changelogs).
Uploaded to Eoan NEW for AA consideration, and now that's Fix COmmitted.
(there's changelog notes for where my bootprints ended up in the code as
well).
** Changed in: ubuntustudio-menu-add
Status: In Progress => Fix
Applied patch, Thank you. Found that "array-based execution" did not
work. The filename was not passed to the executable. So it was changed
back to the working "string-based execution". Yes I am running on the
latest LTS (18.04) and yes it should work there as well as the current
cycle. See line
Note also when I ran this through a test with mixed-indentation like was
used in some places there were some indentation complaints in code
testers. This patch is *large* because it avoids those Indentation
errors that can cause headaches in some cases.
--
You received this bug notification
** Patch added: "Refactor_rm_cp_and_subproccess_calls,_PEP8_style.patch"
https://bugs.launchpad.net/ubuntustudio-menu-add/+bug/1831154/+attachment/5271493/+files/Refactor_rm_cp_and_subproccess_calls%2C_PEP8_style.patch
--
You received this bug notification because you are a member of Ubuntu
The following is a patch containing proposed possible changes.
Note that while the patch LOOKS huge that's because I got rid of tab
indentation with PEP8 dictated 4-spaces indents.
There's some other refactoring done here in order to behave properly
with the new calls for things, replacing 'cmd'
Consider it a goal to remove all the uses of subprocess except exo-
desktop-item-edit, and to replace the string-based execution there with
an array-based execution as seen in the documentation:
https://docs.python.org/3/library/subprocess.html#subprocess.run
For the rm and cp commands:
I had sponsored this, but when digging deeper into the code with
Security Team and Seth Arnold we found some nasty approaches here that
are better handled with Python in-built functions rather than subprocess
wrappers.
I've asked the Archive Admins to reject the upload, and Seth Arnold has
made
Hello.
I went through this with a fine-toothed comb and some additional Lintian
flags against the built binaries.
Only lintian warning was about a missing manpage, but I'm ignoring that
because of the plethora of packages in Ubuntu that don't have manpages.
I didn't notice any major issues with
I took a look at the package. As I indicated in #ubuntustudio-devel,
this has a few things that need fixed.
The BUGS and ROADMAP files are identical. Was this intentional?
Changelog as-is can't be accepted - UNRELEASED is not a valid changelog
target, but I'm probably just being overly
*** This is an automated message ***
This bug is tagged needs-packaging which identifies it as a request for
a new package in Ubuntu. As a part of the managing needs-packaging bug
reports specification,
https://wiki.ubuntu.com/QATeam/Specs/NeedsPackagingBugs, all needs-
packaging bug reports
** Also affects: ubuntu
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Ubuntu
Studio Bugs, which is subscribed to The Ubuntu Studio Project.
Matching subscriptions: UbuntuStudio Bugs
https://bugs.launchpad.net/bugs/1831154
Title:
** Changed in: ubuntustudio-menu-add
Status: New => In Progress
** Changed in: ubuntustudio-menu-add
Assignee: (unassigned) => Erich Eickmeyer (eeickmeyer)
** Changed in: ubuntustudio
Status: New => In Progress
** Changed in: ubuntustudio
Importance: Undecided => Medium
lintian --pedantic returns no errors, subscribing sponsors.
--
You received this bug notification because you are a member of Ubuntu
Studio Bugs, which is subscribed to Ubuntu Studio.
Matching subscriptions: UbuntuStudio Bugs
https://bugs.launchpad.net/bugs/1831154
Title:
[needs-packaging]
14 matches
Mail list logo