aaru-dps/Aaru-aaru-dps
1
0
Fork 0
mirror of https://github.com/aaru-dps/Aaru.git synced 2026-02-04 00:54:33 +00:00
Code Issues 657 Packages Projects Releases 19 Wiki Activity

[PR #280] [MERGED] Fix for the CPCEMU disk images that are not recognized as such #1092

New Issue
Open
opened 2026-01-29 15:41:07 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 15:41:07 +00:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/aaru-dps/Aaru/pull/280
Author: @KailoKyra
Created: 2/12/2020
Status: ✅ Merged
Merged: 2/12/2020
Merged by: @claunia

Base: master ← Head: 279-some-cpcemu-disk-images-are-not-recognized-as-such

📝 Commits (3)

  • b0a6702 Fix the CPM filesystem detection and file listing
  • a8b6ca1 Wrong type used for adding a block when listing the files in a CPM FS
  • 2b402f0 Fix the cpcemu images not recognized as such.

📊 Changes

6 files changed (+13 additions, -14 deletions)

View changed files

📝 DiscImageChef.DiscImages/CPCDSK/Constants.cs (+3 -4)
📝 DiscImageChef.DiscImages/CPCDSK/Identify.cs (+2 -2)
📝 DiscImageChef.DiscImages/CPCDSK/Read.cs (+2 -2)
📝 DiscImageChef.DiscImages/CPCDSK/Structs.cs (+2 -2)
📝 DiscImageChef.Filesystems/CPM/Definitions.cs (+3 -3)
📝 DiscImageChef.Filesystems/CPM/Super.cs (+1 -1)

📄 Description

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • New filesystem, test images in [url]
  • New disc image, test images in [url]
  • New partition scheme, test images in [url]
  • Breaking change (fix or feature that would cause existing functionality to change)
  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Source for the limitation of how much magic bytes are checked in the header :
cpc-wiki
original specification
Closes #279

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/aaru-dps/Aaru/pull/280 **Author:** [@KailoKyra](https://github.com/KailoKyra) **Created:** 2/12/2020 **Status:** ✅ Merged **Merged:** 2/12/2020 **Merged by:** [@claunia](https://github.com/claunia) **Base:** `master` ← **Head:** `279-some-cpcemu-disk-images-are-not-recognized-as-such` --- ### 📝 Commits (3) - [`b0a6702`](https://github.com/aaru-dps/Aaru/commit/b0a670219bbcabf77de13285ad8151212bb966fb) Fix the CPM filesystem detection and file listing - [`a8b6ca1`](https://github.com/aaru-dps/Aaru/commit/a8b6ca13b7cafdfc69087357352200f2cd13a551) Wrong type used for adding a block when listing the files in a CPM FS - [`2b402f0`](https://github.com/aaru-dps/Aaru/commit/2b402f009c363a8dba5a411dce8a47275ea1bb5e) Fix the cpcemu images not recognized as such. ### 📊 Changes **6 files changed** (+13 additions, -14 deletions) <details> <summary>View changed files</summary> 📝 `DiscImageChef.DiscImages/CPCDSK/Constants.cs` (+3 -4) 📝 `DiscImageChef.DiscImages/CPCDSK/Identify.cs` (+2 -2) 📝 `DiscImageChef.DiscImages/CPCDSK/Read.cs` (+2 -2) 📝 `DiscImageChef.DiscImages/CPCDSK/Structs.cs` (+2 -2) 📝 `DiscImageChef.Filesystems/CPM/Definitions.cs` (+3 -3) 📝 `DiscImageChef.Filesystems/CPM/Super.cs` (+1 -1) </details> ### 📄 Description ## Types of changes <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] New filesystem, test images in [url] - [ ] New disc image, test images in [url] - [ ] New partition scheme, test images in [url] - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [x] I have read the **CONTRIBUTING** document. - [x] My code follows the code style of this project. - [ ] My change requires a change to the documentation. - [ ] I have updated the documentation accordingly. - [ ] I have added tests to cover my changes. - [ ] All new and existing tests passed. Source for the limitation of how much magic bytes are checked in the header : [cpc-wiki](http://www.cpcwiki.eu/index.php/Format:DSK_disk_image_file_format ) [original specification](http://www.cpctech.org.uk/docs/dsk.html) Closes #279 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:41:08 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
archive
beginner
bug
bug
dependencies
device
device
dump image format
duplicate
Epic
feature request
filesystem
filter
gui
gui
gui
help wanted
invalid
invalid
media
needs triage
partitioning scheme
pull-request
Mirrored from GitHub Pull Request
 upstream
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aaru-dps/Aaru-aaru-dps#1092
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?