Activity for P.Rouleau

  • P.Rouleau P.Rouleau modified ticket #4

    partclone_imageinfo detects anomalies with image format 0002

  • P.Rouleau P.Rouleau posted a comment on ticket #4

    Yes. Exactly.

  • P.Rouleau P.Rouleau posted a comment on merge request #10

    I was forgetting to point out one point: I am not sure if the modification to src/Makefile.am. I didn't play with makefile for a long time and I am still unfamiliar with Makefile.am features. I tried to define a line like libchecksum_a_SOURCES = libchecksum.c and adding libchecksum.a to a few places but the linker was giving errors. So I ended up adding libchecksum.c to libpartclone_a_SOURCES. I wonder if there is a better solution.

  • P.Rouleau P.Rouleau created merge request #10

    Support image format 0002 in partclone_imageinfo

  • P.Rouleau P.Rouleau modified ticket #3

    imagemount v0.4.2 does not work for all NTFS, many ext4 filesystems

  • P.Rouleau P.Rouleau created ticket #7

    imagemount puts the cache file at the root (/) when the path is not absolute

  • P.Rouleau P.Rouleau created ticket #6

    verify the integrity of the image

  • P.Rouleau P.Rouleau created ticket #5

    Request: partclone_imageinfo should checks if the last sector is missing

  • P.Rouleau P.Rouleau created ticket #4

    partclone_imageinfo detects anomalies with image format 0002

  • P.Rouleau P.Rouleau posted a comment on merge request #9

    I should create a few tickets too. I found a few issues with partclone_imageinfo and I found a few fixes. One of them may affect imagemount and I need to do more tests. I'm playing the the code to better understand it. I have a wip/ branch where I experiment many things. I will split it into many goal focused branches later.

  • P.Rouleau P.Rouleau posted a comment on merge request #9

    OK. All the tabs should be gone now. I finally saw the buttons on the MR above. The "Refresh" button updates the commits to the most recent ones. SourceForge does not seem to do it automatically. And sorry again for my hesitation to go straight to spaces.

  • P.Rouleau P.Rouleau posted a comment on merge request #9

    Sorry, in [1], I proposed to use only spaces for indentation and you agreed and suggested to use a formatting that was close to what is used in partclone itself. Turns out partclone use the mixed indentation too (but not consistent). Unless I did not give priority to the right files? The main reason for me to propose to use only spaces is because I could not configure my editors to follow the mixed rule. However, now that I have found clang, it is possible to configure VS Code with an extension to...

  • P.Rouleau P.Rouleau created merge request #9

    Improve formatting by using clang-format

  • P.Rouleau P.Rouleau modified a comment on merge request #8

    At the risk of "bikeshedding" I just want to note that days ago partclone project updated its README to no longer considers its master branch as unstable , after an issue you opened. Indeed, but he did not delete the release branch itself. So when we look at the existing branches, we see 'master' and 'release' are at different commits, and when we dig deeper, we see that 'release' is related to v0.2.x instead of v0.3.x. So I will do a follow up on that. I think it is best to work with the latest...

  • P.Rouleau P.Rouleau posted a comment on merge request #8

    At the risk of "bikeshedding" I just want to note that days ago partclone project updated its README to no longer considers its master branch as unstable , after an issue you opened. Indeed, but he did not delete the release branch itself. So when we look at the existing branches, we see 'master' and 'release' are at different commits, and when we dig deeper, we see that 'release' is related to v0.2.x instead of v0.3.x. So I will do a follow up on that. I think it is best to work we the latest tag...

  • P.Rouleau P.Rouleau modified a comment on merge request #8

    (Looks like my reply from gmail did not get here, so here it is) In this case, there is the ambition to merge partclone-utils into the partclone itself, so I suggest we imitate partclone's coding style as closely as possible. Partclone uses 4 spaces for indentation (but it not consistent with its use of newline before curly brackets in function implementations). I think it is a good idea to match partclone's coding style. From what I found, .editorconfig's support in Atom and VS Code is pretty limited...

  • P.Rouleau P.Rouleau posted a comment on merge request #8

    (Looks like my reply from gmail did get here, so here it is) In this case, there is the ambition to merge partclone-utils into the partclone itself, so I suggest we imitate partclone's coding style as closely as possible. Partclone uses 4 spaces for indentation (but it not consistent with its use of newline before curly brackets in function implementations). I think it is a good idea to match partclone's coding style. From what I found, .editorconfig's support in Atom and VS Code is pretty limited...

  • P.Rouleau P.Rouleau modified a comment on ticket #3

    On Fri, Feb 19, 2021 at 9:33 PM Rescuezilla rescuezilla@users.sourceforge.net wrote: P.Rouleau wrote https://sourceforge.net/p/partclone-utils/git/merge-requests/8/#f6d4 a comment relevant to this issue on Merge Request #8: One thing I want to try to handle in partclone-utils is to implement a workaround for the missing last sector in NTFS image made by partclone. I still have to play more with the tool to clarify if it is a blocker or just an annoying warning. My response: I don't believe the missing...

  • P.Rouleau P.Rouleau posted a comment on ticket #3

    On Fri, Feb 19, 2021 at 9:33 PM Rescuezilla rescuezilla@users.sourceforge.net wrote: P.Rouleau wrote https://sourceforge.net/p/partclone-utils/git/merge-requests/8/#f6d4 a comment relevant to this issue on Merge Request #8: One thing I want to try to handle in partclone-utils is to implement a workaround for the missing last sector in NTFS image made by partclone. I still have to play more with the tool to clarify if it is a blocker or just an annoying warning. My response: I don't believe the missing...

  • P.Rouleau P.Rouleau posted a comment on merge request #8

    You took me by surprise. I will try to do my part, but I did not expect that honor. 😅 One thing I want to try to handle in partclone-utils is to implement a workaround for the missing last sector in NTFS image made by partclone. I still have to play more with the tool to clarify if it is a blocker or just an annoying warning. One question: how do you feel about the code formatting in partclone-utils? I tried to configure Atom and VS Code to handle the "indent by 4 chars, but tabs are 8 chars", but...

  • P.Rouleau P.Rouleau created merge request #8

    Fix a division by zero with partclone v2 image

  • P.Rouleau P.Rouleau posted a comment on ticket #3

    Hummm, I made this pull request a while ago, 🙄 https://github.com/Thomas-Tsai/partclone/pull/54 I have messaged the Thomas to try to get it merged in the current partclone code.

1