applications/manjaro-architect
3
0
Fork 0
Code Issues 1 Pull requests Projects Releases 39 Packages Wiki Activity Actions

Use -S instead of -Sy pacman flag #580

Merged
yochananmarqos1 merged 1 commit from yochananmarqos1-master-patch-33598 into master 2023-02-18 14:30:47 +00:00
Conversation 13 Commits 1 Files changed 1 +1 -1
yochananmarqos1 commented 2023-02-13 15:08:19 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Warning: When installing packages in Arch, avoid refreshing the package list without upgrading the system (for example, when a package is no longer found in the official repositories). In practice, do not run pacman -Sy package_name instead of pacman -Syu package_name, as this could lead to dependency issues. See System maintenance#Partial upgrades are unsupported and BBS#89328.

-- https://wiki.archlinux.org/title/Pacman#Installing_packages

> Warning: When installing packages in Arch, avoid refreshing the package list without [upgrading the system](https://wiki.archlinux.org/title/Pacman#Upgrading_packages) (for example, when a [package is no longer found](https://wiki.archlinux.org/title/Pacman#Packages_cannot_be_retrieved_on_installation) in the official repositories). In practice, do **not** run pacman -Sy _package_name_ instead of pacman -Sy**u** package_name, as this could lead to dependency issues. See [System maintenance#Partial upgrades are unsupported](https://wiki.archlinux.org/title/System_maintenance#Partial_upgrades_are_unsupported) and [BBS#89328](https://bbs.archlinux.org/viewtopic.php?id=89328). -- https://wiki.archlinux.org/title/Pacman#Installing_packages
0xbilko commented 2023-02-18 11:06:44 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Hi Mark!
This patch breaks the installer launch. See the screenshot https://i.imgur.com/rRNX0Pw.png.
This patch is already used to build the package (gitlab.manjaro.org/packages/extra/manjaro-architect/-@1a46b46466 (9b9baac1eb9b72790eef5540a1685306fc43fd6c_21_20)) and it breaks the script. If we change the concept, it should not break the script.

Hi Mark! This patch breaks the installer launch. See the screenshot https://i.imgur.com/rRNX0Pw.png. This patch is already used to build the package (https://gitlab.manjaro.org/packages/extra/manjaro-architect/-/commit/1a46b46466f86c55f694d1f4b49330f918d440ca#9b9baac1eb9b72790eef5540a1685306fc43fd6c_21_20) and it breaks the script. If we change the concept, it should not break the script.
0xbilko commented 2023-02-18 11:27:10 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

My understanding is that we should use pacman -Syu instead of pacman -Sy or pacman -S.

My understanding is that we should use `pacman -Syu` instead of `pacman -Sy` or `pacman -S`.
pheiduck commented 2023-02-18 14:19:50 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

approved this merge request

approved this merge request
pheiduck commented 2023-02-18 14:27:15 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Yes, we need to drop the patch on the PKGBUILD if this PR gets merged, so this happend because you try to apply the Patch twice.

Yes, we need to drop the patch on the PKGBUILD if this PR gets merged, so this happend because you try to apply the Patch twice.
pheiduck commented 2023-02-18 14:30:47 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

mentioned in commit 96e042eac4

mentioned in commit 96e042eac433684e73876b4db72560756a1aa585
pheiduck (Migrated from gitlab2.manjaro.org) merged commit 96e042eac4 into master 2023-02-18 14:30:48 +00:00
pheiduck commented 2023-02-18 14:49:22 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

As described in the Arch Linux wiki we should use pacman -S. - Will be merged / Included in 0.9.37.

As described in the Arch Linux wiki we should use `pacman -S`. - Will be merged / Included in 0.9.37.
0xbilko commented 2023-02-18 16:56:34 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

@pheiduck wtf? This patch in this form breaks the script. I wrote about it and you just ignored me and merged patch into master...

@pheiduck wtf? This patch in this form breaks the script. I wrote about it and you just ignored me and merged patch into master...
0xbilko commented 2023-02-18 17:06:35 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

@yochananmarqos1 @pheiduck I changed to pacman -Syu(gitlab.manjaro.org/applications/manjaro-architect/-@50fc091585) and recreated the tag 0.9.37.
Changes should not break the script. If you don't agree with my commit, then let's have a discussion about it.

@yochananmarqos1 @pheiduck I changed to `pacman -Syu`(https://gitlab.manjaro.org/applications/manjaro-architect/-/commit/50fc09158557b13dd21fc43a635e939f97b59e0f) and recreated the tag `0.9.37`. Changes should not break the script. If you don't agree with my commit, then let's have a discussion about it.
yochananmarqos1 commented 2023-02-18 18:12:07 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

@beliys Turns out I was wrong to do this to begin with, sorry about the trouble. I'll rebuild and push 0.9.37-2.

The archinstall script also uses -Sy on the live ISO.

@beliys Turns out I was wrong to do this to begin with, sorry about the trouble. I'll rebuild and push 0.9.37-2. The `archinstall` script [also uses `-Sy`](https://github.com/archlinux/archinstall#using-a-live-iso-image) on the live ISO.
pheiduck commented 2023-02-18 23:44:06 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

@beliys okay, than it's fine to revert it. - I am sorry, for confuse

@beliys okay, than it's fine to revert it. - I am sorry, for confuse
0xbilko commented 2023-02-20 08:37:58 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Unfortunately, we also get errors from pacman -Syu, and if it turned out not to be critical, then I returned pacman -Sy, which had no errors.

@yochananmarqos1, сan you build a package with the tag 0.9.38?

Unfortunately, we also get errors from `pacman -Syu`, and if it turned out not to be critical, then I returned `pacman -Sy`, which had no errors. @yochananmarqos1, сan you build a package with the tag 0.9.38?
yochananmarqos1 commented 2023-02-20 15:37:21 +00:00 (Migrated from gitlab2.manjaro.org)
Copy link

Done ✅

[Done](https://gitlab.manjaro.org/packages/extra/manjaro-architect/-/commit/6380ecd41ef03e7882d0c832d2b61945e8824a5b) :white_check_mark:
pheiduck (Migrated from gitlab2.manjaro.org) approved these changes 2026-09-09 10:30:43 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
pheiduck
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
applications/manjaro-architect!580
No description provided.